Skip to content

Commit

Permalink
fix(ci): Also test iroh-cli for cargo features (#2122)
Browse files Browse the repository at this point in the history
## Description

We really ought to use something like cargo-metadata to pull this
information from the cargo.toml file.  Maybe one day one of us will be
motivated enough to do this.

## Notes & open questions

<!-- Any notes, remarks or open questions you have to make about the PR.
-->

## Change checklist

- [x] Self-review.
- [x] Documentation updates if relevant.
- [x] Tests if relevant.
  • Loading branch information
flub authored Mar 25, 2024
1 parent 8dcb196 commit a136b1d
Showing 1 changed file with 7 additions and 2 deletions.
9 changes: 7 additions & 2 deletions .github/workflows/tests.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ env:
RUSTFLAGS: -Dwarnings
RUSTDOCFLAGS: -Dwarnings
SCCACHE_CACHE_SIZE: "50G"
CRATES_LIST: "iroh,iroh-bytes,iroh-gossip,iroh-metrics,iroh-net,iroh-sync,iroh-test"
CRATES_LIST: "iroh,iroh-bytes,iroh-gossip,iroh-metrics,iroh-net,iroh-sync,iroh-test,iroh-cli"

jobs:
build_and_test_nix:
Expand Down Expand Up @@ -92,7 +92,12 @@ jobs:
for i in ${CRATES_LIST//,/ }
do
echo "Checking $i $FEATURES"
cargo check -p $i $FEATURES --lib --bins
if [ $i = "iroh-cli" ]; then
targets="--bins"
else
targets="--lib --bins"
fi
cargo check -p $i $FEATURES $targets
done
env:
RUST_LOG: ${{ runner.debug && 'TRACE' || 'DEBUG'}}
Expand Down

0 comments on commit a136b1d

Please sign in to comment.