ci: check and lint with all features
The facade crate enables nothing by default, so cargo check and cargo clippy without --all-features skipped almost all of its code and every example. A deliberate mistake placed in feature gated facade code was invisible to the old commands and caught by the new ones.
This commit is contained in:
parent
fac3732f08
commit
f34ca07e88
|
|
@ -18,11 +18,13 @@ jobs:
|
||||||
- name: Format
|
- name: Format
|
||||||
run: cargo fmt --all --check
|
run: cargo fmt --all --check
|
||||||
- name: Check
|
- name: Check
|
||||||
run: cargo check --workspace --all-targets
|
# --all-features matters: the facade crate enables nothing by default,
|
||||||
|
# so without it almost all of its code, and every example, is skipped.
|
||||||
|
run: cargo check --workspace --all-targets --all-features
|
||||||
- name: Clippy
|
- name: Clippy
|
||||||
run: cargo clippy --workspace --all-targets -- -D warnings
|
run: cargo clippy --workspace --all-targets --all-features -- -D warnings
|
||||||
- name: Test
|
- name: Test
|
||||||
run: cargo test --workspace
|
run: cargo test --workspace --all-features
|
||||||
- name: Integration checks
|
- name: Integration checks
|
||||||
# The cross-layer checks are examples rather than tests, because they
|
# The cross-layer checks are examples rather than tests, because they
|
||||||
# are also meant to be read and run by hand. `cargo test` does not run
|
# are also meant to be read and run by hand. `cargo test` does not run
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue