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 <[email protected]>
This commit is contained in:
Henry Park
2026-03-09 18:44:57 -07:00
committed by GitHub
co-authored by Claude Opus 4.6
parent a5f88b32fd
commit 7de639e782
2 changed files with 8 additions and 4 deletions
+1 -1
View File
@@ -2,7 +2,7 @@ name: Claude Code Review
on:
pull_request:
types: [opened, labeled]
types: [labeled]
permissions:
contents: read
+7 -3
View File
@@ -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