Skip to content

Commit

Permalink
downgrade clippy to stable until clap-rs/clap#4733 is fixed
Browse files Browse the repository at this point in the history
  • Loading branch information
ralexstokes committed Feb 28, 2023
1 parent ce3cba3 commit 66c47be
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 3 deletions.
9 changes: 7 additions & 2 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@ jobs:
- name: Install toolchain
uses: dtolnay/rust-toolchain@nightly
with:
components: rustfmt, clippy
components: rustfmt

- name: Rust cache
uses: Swatinem/rust-cache@v2
Expand All @@ -61,5 +61,10 @@ jobs:
- name: Check format
run: cargo +nightly fmt --all --check

# TODO remove after `clap` is fixed
- name: Install stable clippy
run: rustup component add clippy # --toolchain 1.67-x86_64-unknown-linux-gnu

- name: Check clippy
run: cargo +nightly clippy --all-targets --all-features --all --verbose -- -D warnings
# TODO restore +nightly after fixing this: https://github.com/clap-rs/clap/issues/4733
run: cargo clippy --all-targets --all-features --all --verbose -- -D warnings
3 changes: 2 additions & 1 deletion justfile
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,8 @@ test:
fmt:
cargo +nightly fmt --all
lint: fmt validate-example-config
cargo +nightly clippy --all-targets --all-features --all
# TODO restore +nightly after fixing this: https://github.com/clap-rs/clap/issues/4733
cargo clippy --all-targets --all-features --all
build:
cargo build --all-targets --all-features --all
run-ci: lint build test run-integration-tests

0 comments on commit 66c47be

Please sign in to comment.