Skip to content

Commit

Permalink
fix: modified ci to run bls feature tests
Browse files Browse the repository at this point in the history
  • Loading branch information
shreyas-londhe committed Jan 5, 2025
1 parent fb7ea93 commit 48c6c34
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 4 deletions.
6 changes: 3 additions & 3 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ jobs:
# Running `rustc` or Cargo should automatically install the toolchain specified in `rust-toolchain.toml`.
- uses: Swatinem/rust-cache@v2
- name: Run cargo build
run: cargo build --release --features default-networks
run: cargo build --release --features default-networks --features bls-blst
- name: Check if code is formatted (Linux)
if: runner.os == 'Linux'
run: cargo fmt --check
Expand All @@ -34,10 +34,10 @@ jobs:
# It's a known issue that networking tests randomly fails on Macos
- name: Run tests
if: runner.os == 'Macos'
run: cargo test --release --no-fail-fast -- --skip behaviour --skip common
run: cargo test --release --no-fail-fast --features bls-blst -- --skip behaviour --skip common
- name: Run tests
if: runner.os != 'Macos'
run: cargo test --release --no-fail-fast
run: cargo test --release --no-fail-fast --features bls-zkcrypto
- name: Check consensus-spec-tests coverage (Linux)
if: runner.os == 'Linux'
run: scripts/ci/consensus-spec-tests-coverage.rb
1 change: 0 additions & 1 deletion bls/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@ edition = { workspace = true }
authors = ["Grandine <[email protected]>"]

[features]
default = ["bls-blst"]
bls-blst = [
"dep:blst",
"dep:itertools"
Expand Down
1 change: 1 addition & 0 deletions scripts/ci/clippy.bash
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@ options=(
--profile test

--features eth2-cache
--features bls-blst

# TODO(Grandine Team): Clean up `dedicated_executor` and `eth2_libp2p`.
# Go back to linting all packages implicitly.
Expand Down

0 comments on commit 48c6c34

Please sign in to comment.