Skip to content

Commit

Permalink
Clean-up GitHub CI build job and submodule init (#1132)
Browse files Browse the repository at this point in the history
* CI: use actions/checkout to initialize submodules

* adjust GitHub Actions build CI job
  • Loading branch information
Robbepop authored Jul 19, 2024
1 parent e355a14 commit 098508b
Showing 1 changed file with 38 additions and 37 deletions.
75 changes: 38 additions & 37 deletions .github/workflows/rust.yml
Original file line number Diff line number Diff line change
Expand Up @@ -26,43 +26,45 @@ jobs:
run: cargo build --workspace --locked --verbose
- name: Build (all features)
run: cargo build --workspace --locked --all-features --verbose
- name: Build (no_std + no-hash-maps)
run: cargo build --locked -p wasmi_collections --no-default-features --features no-hash-maps --verbose
- name: Build (no_std)
run: >-
cargo build
--workspace
--package wasmi
--locked
--lib
--no-default-features
--target x86_64-unknown-none
--verbose
- name: Build (no_std + no-hash-maps)
run: >-
cargo build
--package wasmi
--locked
--lib
--no-default-features
--features no-hash-maps
--target x86_64-unknown-none
--verbose
--exclude wasmi_cli
--exclude wasmi_wasi
--exclude wasmi_fuzz
- name: Build (wasm32)
- name: Build (no_std + wasm32)
run: >-
cargo build
--workspace
--locked
--lib
--no-default-features
--target wasm32-unknown-unknown
--verbose
--exclude wasmi_cli
--exclude wasmi_wasi
--exclude wasmi_fuzz
cargo build
--package wasmi
--locked
--lib
--no-default-features
--target wasm32-unknown-unknown
--verbose
test-asan:
name: Test (Address Sanitizer)
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@692973e3d937129bcbf40652eb9f2f61becf3332
with:
submodules: true
- uses: dtolnay/rust-toolchain@nightly
- name: Set up Rust cache
uses: Swatinem/rust-cache@v2
- name: Checkout Submodules
run: git submodule update --init --recursive
- name: Show Rust Toolchain
run: rustup show
- name: "Add Rust Component: `rust-src`"
Expand All @@ -84,11 +86,11 @@ jobs:
runs-on: ${{ matrix.os }}
steps:
- uses: actions/checkout@692973e3d937129bcbf40652eb9f2f61becf3332
with:
submodules: true
- uses: dtolnay/rust-toolchain@stable
- name: Set up Rust cache
uses: Swatinem/rust-cache@v2
- name: Checkout Submodules
run: git submodule update --init --recursive
- name: Build (default features)
env:
RUSTFLAGS: "--cfg debug_assertions"
Expand Down Expand Up @@ -153,11 +155,11 @@ jobs:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@692973e3d937129bcbf40652eb9f2f61becf3332
with:
submodules: true
- uses: dtolnay/rust-toolchain@nightly
- name: Set up Rust cache
uses: Swatinem/rust-cache@v2
- name: Checkout Submodules
run: git submodule update --init --recursive
- name: Install cargo-udeps
run: |
# Note: We use `|| true` because cargo install returns an error
Expand All @@ -171,15 +173,15 @@ jobs:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@692973e3d937129bcbf40652eb9f2f61becf3332
with:
submodules: true
- uses: dtolnay/rust-toolchain@nightly
- name: Set up Rust cache
uses: Swatinem/rust-cache@v2
with:
cache-directories: |
~/fuzz/corpus/translate/
~/fuzz/corpus/translate_metered/
- name: Checkout Submodules
run: git submodule update --init --recursive
- name: Install cargo-fuzz
run: |
# Note: We use `|| true` because cargo install returns an error
Expand All @@ -197,14 +199,14 @@ jobs:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@692973e3d937129bcbf40652eb9f2f61becf3332
with:
submodules: true
- uses: dtolnay/rust-toolchain@nightly
- name: Set up Rust cache
uses: Swatinem/rust-cache@v2
with:
cache-directories: |
~/fuzz/corpus/execute/
- name: Checkout Submodules
run: git submodule update --init --recursive
- name: Install cargo-fuzz
run: |
# Note: We use `|| true` because cargo install returns an error
Expand All @@ -220,14 +222,14 @@ jobs:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@692973e3d937129bcbf40652eb9f2f61becf3332
with:
submodules: true
- uses: dtolnay/rust-toolchain@nightly
- name: Set up Rust cache
uses: Swatinem/rust-cache@v2
with:
cache-directories: |
~/fuzz/corpus/differential/
- name: Checkout Submodules
run: git submodule update --init --recursive
- name: Install cargo-fuzz
run: |
# Note: We use `|| true` because cargo install returns an error
Expand All @@ -243,14 +245,14 @@ jobs:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@692973e3d937129bcbf40652eb9f2f61becf3332
with:
submodules: true
- uses: dtolnay/rust-toolchain@nightly
with:
components: miri
targets: x86_64-unknown-linux-gnu
- name: Set up Rust cache
uses: Swatinem/rust-cache@v2
- name: Checkout Submodules
run: git submodule update --init --recursive
- name: Install cargo-nextest
run: |
# Note: We use `|| true` because cargo install returns an error
Expand All @@ -268,14 +270,14 @@ jobs:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@692973e3d937129bcbf40652eb9f2f61becf3332
with:
submodules: true
- uses: dtolnay/rust-toolchain@nightly
with:
components: miri
targets: x86_64-unknown-linux-gnu
- name: Set up Rust cache
uses: Swatinem/rust-cache@v2
- name: Checkout Submodules
run: git submodule update --init --recursive
- name: Install cargo-nextest
run: |
# Note: We use `|| true` because cargo install returns an error
Expand All @@ -294,13 +296,13 @@ jobs:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@692973e3d937129bcbf40652eb9f2f61becf3332
with:
submodules: true
- uses: dtolnay/rust-toolchain@nightly
with:
components: clippy
- name: Set up Rust cache
uses: Swatinem/rust-cache@v2
- name: Checkout Submodules
run: git submodule update --init --recursive
- name: Clippy (default features)
run: cargo clippy --workspace --locked -- -D warnings
- name: Clippy (all features)
Expand All @@ -321,12 +323,11 @@ jobs:
- name: Checkout repository
uses: actions/checkout@692973e3d937129bcbf40652eb9f2f61becf3332
with:
submodules: true
fetch-depth: 0
- uses: dtolnay/rust-toolchain@stable
- name: Set up Rust cache
uses: Swatinem/rust-cache@v2
- name: Checkout Submodules
run: git submodule update --init --recursive
- name: Run cargo-tarpaulin (default features)
uses: actions-rs/[email protected]
with:
Expand Down

0 comments on commit 098508b

Please sign in to comment.