Skip to content

Commit 58194f4

Browse files
authored
Add some more parallelism to CI jobs (#1545)
* Add some more parallelism to CI jobs Split out some work from the main "test the workspace" CI job to avoid bottlenecking too much on that. Notably split the various extra builds/tests into their own jobs and move the C API build/run into its own job. * Name the C API job differently
1 parent c224461 commit 58194f4

1 file changed

Lines changed: 28 additions & 2 deletions

File tree

.github/workflows/main.yml

Lines changed: 28 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -85,11 +85,33 @@ jobs:
8585
- name: Install Rust (rustup)
8686
run: rustup update ${{ matrix.rust }} --no-self-update && rustup default ${{ matrix.rust }}
8787
- run: cargo test --locked --all
88+
89+
test_extra_features:
90+
name: Test with extra Cargo features
91+
runs-on: ubuntu-latest
92+
steps:
93+
- uses: actions/checkout@v4
94+
with:
95+
submodules: true
96+
- run: rustup update stable --no-self-update && rustup default stable
8897
- run: cargo test --locked -p wasmparser --benches
8998
- run: cargo test --locked -p wasm-encoder --all-features
9099
- run: cargo test -p wasm-smith --features wasmparser
91-
- run: cargo build --manifest-path crates/wast/Cargo.toml --no-default-features
92-
- run: cargo build --manifest-path crates/wast/Cargo.toml --no-default-features --features wasm-module
100+
101+
test_capi:
102+
name: Test the C API
103+
runs-on: ${{ matrix.os }}
104+
strategy:
105+
matrix:
106+
include:
107+
- os: ubuntu-latest
108+
- os: macos-latest
109+
- os: windows-latest
110+
steps:
111+
- uses: actions/checkout@v4
112+
with:
113+
submodules: true
114+
- run: rustup update stable --no-self-update && rustup default stable
93115
- run: cmake -S examples -B examples/build -DCMAKE_BUILD_TYPE=Release
94116
- run: cmake --build examples/build --config Release
95117

@@ -191,6 +213,8 @@ jobs:
191213
- run: cargo check --no-default-features -p wasmparser --target x86_64-unknown-none
192214
- run: cargo check --no-default-features -p wasmparser --features std
193215
- run: cargo check --no-default-features -p wasmparser --features validate
216+
- run: cargo check --no-default-features -p wast
217+
- run: cargo check --no-default-features -p wast --features wasm-module
194218
- run: |
195219
if cargo tree -p wasm-smith --no-default-features -e no-dev | grep wasmparser; then
196220
echo wasm-smith without default features should not depend on wasmparser
@@ -231,6 +255,8 @@ jobs:
231255
- doc
232256
- build
233257
- verify-publish
258+
- test_capi
259+
- test_extra_features
234260
if: always()
235261

236262
steps:

0 commit comments

Comments
 (0)