Skip to content

Commit

Permalink
CI: use stable clippy
Browse files Browse the repository at this point in the history
Using nightly one leads to having to resolve lints more frequently than rust releases, which is
getting inconvenient.

It also leads to a small compatibility issue with stable clippy: if we need to #[allow] a lint that
is only present in nightly clippy, then stable clippy complains about unknown ling in the `allow`.
  • Loading branch information
strohel committed Dec 1, 2022
1 parent eac6071 commit 00c2b3c
Showing 1 changed file with 13 additions and 2 deletions.
15 changes: 13 additions & 2 deletions .github/workflows/rust.yml
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ jobs:
command: test
args: --features v6-test --verbose

fmt-and-clippy:
fmt:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v2
Expand All @@ -56,12 +56,23 @@ jobs:
profile: minimal
toolchain: nightly
override: true
components: rustfmt, clippy
components: rustfmt
- name: Rustfmt
uses: actions-rs/cargo@v1
with:
command: fmt
args: --all -- --check

clippy:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v2
- uses: actions-rs/toolchain@v1
with:
profile: minimal
toolchain: stable
override: true
components: clippy
- name: Clippy
uses: actions-rs/cargo@v1
with:
Expand Down

0 comments on commit 00c2b3c

Please sign in to comment.