Merge rust-bitcoin/rust-bitcoin#3394: Add stricter doc test to justfile
bd32d1db78
Add stricter doc test to justfile (Jamil Lambert, PhD) Pull request description: In order to show warnings for unused imports etc. in doc examples as mentioned in #3362, an extra test has been added to the justfile for `just lint` and the doc test in `just sane` has been removed since it calls `just lint`. ACKs for top commit: apoelstra: ACKbd32d1db78
successfully ran local tests tcharding: ACKbd32d1db78
Tree-SHA512: 088244aa9cd07e4aab197d1ad675c6ed778f10cec5304bfcecdd9a0526cd6395f14db69e71a464acd1e41ce54610a51257481602348f3ee65b0228550cc6e12c
This commit is contained in:
commit
13b8446c1c
5
justfile
5
justfile
|
@ -12,6 +12,8 @@ check:
|
||||||
# Lint everything.
|
# Lint everything.
|
||||||
lint:
|
lint:
|
||||||
cargo +$(cat ./nightly-version) clippy --workspace --all-targets --all-features -- --deny warnings
|
cargo +$(cat ./nightly-version) clippy --workspace --all-targets --all-features -- --deny warnings
|
||||||
|
# lint warnings get inhibited unless we use `--nocapture`
|
||||||
|
cargo test --quiet --workspace --doc -- --nocapture
|
||||||
|
|
||||||
# Run cargo fmt
|
# Run cargo fmt
|
||||||
fmt:
|
fmt:
|
||||||
|
@ -31,9 +33,6 @@ sane: lint
|
||||||
cargo test --quiet --workspace --all-targets > /dev/null || exit 1
|
cargo test --quiet --workspace --all-targets > /dev/null || exit 1
|
||||||
cargo test --quiet --workspace --all-targets --all-features > /dev/null || exit 1
|
cargo test --quiet --workspace --all-targets --all-features > /dev/null || exit 1
|
||||||
|
|
||||||
# doctests don't get run from workspace root with `cargo test`.
|
|
||||||
cargo test --quiet --workspace --doc || exit 1
|
|
||||||
|
|
||||||
# Make an attempt to catch feature gate problems in doctests
|
# Make an attempt to catch feature gate problems in doctests
|
||||||
cargo test --manifest-path bitcoin/Cargo.toml --doc --no-default-features > /dev/null || exit 1
|
cargo test --manifest-path bitcoin/Cargo.toml --doc --no-default-features > /dev/null || exit 1
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue