Skip to content

Commit

Permalink
Split apart build and test CI jobs
Browse files Browse the repository at this point in the history
Originally, the tests required the binary produced by 'cargo build', but
now that that's fixed, and the tests require code coverage
instrumentation (forcing a rebuild anyways), it's better to do the build
and test in parallel.
  • Loading branch information
Notgnoshi committed Dec 25, 2024
1 parent 409d9f3 commit ac7242f
Showing 1 changed file with 16 additions and 4 deletions.
20 changes: 16 additions & 4 deletions .github/workflows/lint.yml
Original file line number Diff line number Diff line change
Expand Up @@ -44,6 +44,22 @@ jobs:
run: cargo fmt -- --check --config group_imports=StdExternalCrate,imports_granularity=Module

build:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
- name: Setup Rust toolchain
uses: dtolnay/rust-toolchain@master
with:
toolchain: stable
components: clippy
- name: Setup Rust cache
uses: swatinem/rust-cache@v2
- name: Build
run: cargo build --all-targets --all-features
- name: Clippy
run: cargo clippy --no-deps --all-targets --all-features

test:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
Expand All @@ -58,10 +74,6 @@ jobs:
uses: taiki-e/install-action@v2
with:
tool: cargo-nextest,cargo-llvm-cov
- name: Build
run: cargo build --all-targets --all-features
- name: Clippy
run: cargo clippy --no-deps --all-targets --all-features
- name: Test
run: |
git fetch
Expand Down

0 comments on commit ac7242f

Please sign in to comment.