Skip to content

Commit a8f4c8c

Browse files
committed
remove --all from cargo test
according to the help the option is deprecated and equivalent to `--workspace` and we haven't a workspace. Moreover, and most importantly, the option somehow activates features which should not be present. for example: `cargo tree --no-default-features -e no-dev --all | grep serde` has dependencies in the output while without `--all` returns nothing
1 parent 0a78d54 commit a8f4c8c

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

contrib/test.sh

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -20,11 +20,11 @@ fi
2020
if [ "$DO_FEATURE_MATRIX" = true ]
2121
then
2222
# Test without any features first
23-
cargo test --all --verbose --no-default-features
23+
cargo test --verbose --no-default-features
2424
# Then test with the default features
25-
cargo test --all --verbose
25+
cargo test --verbose
2626
# Then test with the default features
27-
cargo test --all --all-features --verbose
27+
cargo test --all-features --verbose
2828

2929
# Also build and run each example to catch regressions
3030
cargo build --examples

0 commit comments

Comments
 (0)