Skip to content

Commit d032a4d

Browse files
committed
ci: check fmt and clippy on template
I'm concerned clippy don't skip the cargo-gpu install step, increasing lint ci times. But that's not our bottleneck either
1 parent cbed0de commit d032a4d

File tree

1 file changed

+7
-5
lines changed

1 file changed

+7
-5
lines changed

.github/workflows/ci.yaml

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -82,18 +82,18 @@ jobs:
8282
# to update the branch protection rules when the test matrix changes
8383
test_success:
8484
runs-on: ubuntu-24.04
85-
needs: [test, xtask-test, lint]
85+
needs: [test, template-test, lint]
8686
# Hack for buggy GitHub Actions behavior with skipped checks: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/collaborating-on-repositories-with-code-quality-features/troubleshooting-required-status-checks#handling-skipped-but-required-checks
8787
if: ${{ always() }}
8888
steps:
8989
# Another hack is to actually check the status of the dependencies or else it'll fall through
9090
- run: |
9191
echo "Checking statuses..."
9292
[[ "${{ needs.test.result }}" == "success" ]] || exit 1
93-
[[ "${{ needs.xtask-test.result }}" == "success" ]] || exit 1
93+
[[ "${{ needs.template-test.result }}" == "success" ]] || exit 1
9494
[[ "${{ needs.lint.result }}" == "success" ]] || exit 1
9595
96-
xtask-test:
96+
template-test:
9797
name: xtask test & lint
9898
runs-on: ubuntu-24.04
9999
steps:
@@ -116,10 +116,12 @@ jobs:
116116
run: cargo fetch --locked
117117
- name: xtask test
118118
run: cargo nextest run -p xtask
119-
- name: xtask fmt
120-
run: cargo fmt -p xtask --all -- --check
119+
- name: fmt
120+
run: cargo fmt --all -- --check
121121
- name: xtask clippy
122122
run: cargo clippy -p xtask --all-targets -- -D warnings
123+
- name: template clippy
124+
run: cargo clippy --workspace --exclude xtask --all-targets -- -D warnings
123125

124126
lint:
125127
name: lint

0 commit comments

Comments
 (0)