Skip to content

Commit

Permalink
GitHub: Move fuzz target checking to the stable Rust toolchain job
Browse files Browse the repository at this point in the history
...because the fuzz target doesn't have a lockfile, and crates don't
generally support arbitrary nightly versions. See
dtolnay/proc-macro2#307 for an example.
  • Loading branch information
jrose-signal committed Dec 10, 2021
1 parent 5104d19 commit 974b7d2
Showing 1 changed file with 9 additions and 6 deletions.
15 changes: 9 additions & 6 deletions .github/workflows/build_and_test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -104,6 +104,15 @@ jobs:
- name: Build benches
run: cargo +stable build --workspace --benches --verbose

- name: Check that the fuzz target still builds
# We do this on stable because the fuzz target doesn't have a lockfile,
# and crates don't generally support arbitrary nightly versions.
# See https://github.com/dtolnay/proc-macro2/issues/307 for an example.
run: cargo +stable check --all-targets
working-directory: rust/protocol/fuzz
env:
RUSTFLAGS: --cfg fuzzing

rust-nightly:
name: Rust nightly

Expand Down Expand Up @@ -154,12 +163,6 @@ jobs:
- name: Clippy
run: cargo clippy --workspace --all-targets -- -D warnings

- name: Clippy (protocol fuzzing)
run: cargo clippy --all-targets -- -D warnings
working-directory: rust/protocol/fuzz
env:
RUSTFLAGS: --cfg fuzzing

java:
name: Java

Expand Down

0 comments on commit 974b7d2

Please sign in to comment.