Remove obsolete clippy scripts.
Now that clippy is installable via rustup and is generally more stable, we no longer need special scripts. `rustup component add clippy-preview` is sufficient to install clippy, and `cargo clippy` is sufficient to run it. Also, don't run clippy in test-all.sh. We do generally want to fix things clippy reports, however it's not a requirement that the code be kept clippy-warning-free at all times.
This commit is contained in:
@@ -1,7 +0,0 @@
|
|||||||
#!/bin/bash
|
|
||||||
set -euo pipefail
|
|
||||||
|
|
||||||
# Check all sources with clippy.
|
|
||||||
# In the clif-util crate (root dir) clippy will only work with nightly cargo -
|
|
||||||
# there is a bug where it will reject the commands passed to it by cargo 0.25.0
|
|
||||||
exec cargo +nightly clippy --all
|
|
||||||
10
test-all.sh
10
test-all.sh
@@ -8,7 +8,7 @@ set -euo pipefail
|
|||||||
# - Make a release build.
|
# - Make a release build.
|
||||||
# - Run unit tests for all Rust crates (including the filetests)
|
# - Run unit tests for all Rust crates (including the filetests)
|
||||||
# - Build API documentation.
|
# - Build API documentation.
|
||||||
# - Optionally, run clippy and fuzzing.
|
# - Optionally, run fuzzing.
|
||||||
#
|
#
|
||||||
# All tests run by this script should be passing at all times.
|
# All tests run by this script should be passing at all times.
|
||||||
|
|
||||||
@@ -51,14 +51,6 @@ cargo test --all
|
|||||||
banner "Rust documentation: $topdir/target/doc/wasmtime/index.html"
|
banner "Rust documentation: $topdir/target/doc/wasmtime/index.html"
|
||||||
cargo doc
|
cargo doc
|
||||||
|
|
||||||
# Run clippy if we have it.
|
|
||||||
banner "Rust linter"
|
|
||||||
if "$topdir/check-clippy.sh"; then
|
|
||||||
"$topdir/clippy-all.sh"
|
|
||||||
else
|
|
||||||
echo "\`cargo +nightly install clippy\` for optional rust linting"
|
|
||||||
fi
|
|
||||||
|
|
||||||
# Ensure fuzzer works by running it with a single input
|
# Ensure fuzzer works by running it with a single input
|
||||||
# Note LSAN is disabled due to https://github.com/google/sanitizers/issues/764
|
# Note LSAN is disabled due to https://github.com/google/sanitizers/issues/764
|
||||||
banner "cargo fuzz check"
|
banner "cargo fuzz check"
|
||||||
|
|||||||
Reference in New Issue
Block a user