From 7de639e7824f961add5324879e9295b87c56d0d2 Mon Sep 17 00:00:00 2001 From: Henry Park Date: Mon, 9 Mar 2026 18:44:57 -0700 Subject: [PATCH] fix(ci): cherry-pick fmt + clippy for staging PRs [skip-regression-check] (#803) Cherry-pick of #802: run fmt + clippy on staging PRs, skip Windows clippy, simplify claude-review trigger to labeled-only. Co-authored-by: Claude Opus 4.6 --- .github/workflows/claude-review.yml | 2 +- .github/workflows/code_style.yml | 10 +++++++--- 2 files changed, 8 insertions(+), 4 deletions(-) diff --git a/.github/workflows/claude-review.yml b/.github/workflows/claude-review.yml index 24d2fe98..3836a5f9 100644 --- a/.github/workflows/claude-review.yml +++ b/.github/workflows/claude-review.yml @@ -2,7 +2,7 @@ name: Claude Code Review on: pull_request: - types: [opened, labeled] + types: [labeled] permissions: contents: read diff --git a/.github/workflows/code_style.yml b/.github/workflows/code_style.yml index c65aa0df..620760ae 100644 --- a/.github/workflows/code_style.yml +++ b/.github/workflows/code_style.yml @@ -1,8 +1,6 @@ name: Code Style on: pull_request: - branches: - - main jobs: format: @@ -46,6 +44,7 @@ jobs: clippy-windows: name: Clippy Windows (${{ matrix.name }}) + if: github.base_ref == 'main' runs-on: windows-latest strategy: fail-fast: false @@ -78,7 +77,12 @@ jobs: needs: [format, clippy, clippy-windows] steps: - run: | - if [[ "${{ needs.format.result }}" != "success" || "${{ needs.clippy.result }}" != "success" || "${{ needs.clippy-windows.result }}" != "success" ]]; then + if [[ "${{ needs.format.result }}" != "success" || "${{ needs.clippy.result }}" != "success" ]]; then echo "One or more jobs failed" exit 1 fi + # clippy-windows only runs on main PRs, so skip/success are both acceptable + if [[ "${{ needs.clippy-windows.result }}" == "failure" ]]; then + echo "Windows clippy failed" + exit 1 + fi