From d7d63892568f5693852e3d0bbdaf690aaafb4bc8 Mon Sep 17 00:00:00 2001 From: Pascal Seitz Date: Fri, 6 Oct 2023 15:40:44 +0800 Subject: [PATCH 1/2] run coverage only after merge coverage is a quite slow step in CI. It can be run only after merging --- .github/workflows/coverage.yml | 2 -- .github/workflows/test.yml | 9 +++++++++ 2 files changed, 9 insertions(+), 2 deletions(-) diff --git a/.github/workflows/coverage.yml b/.github/workflows/coverage.yml index 72c334de67..e1483f6693 100644 --- a/.github/workflows/coverage.yml +++ b/.github/workflows/coverage.yml @@ -3,8 +3,6 @@ name: Coverage on: push: branches: [main] - pull_request: - branches: [main] # Ensures that we cancel running jobs for the same PR / same workflow. concurrency: diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml index c2ae025c2b..64d413acbf 100644 --- a/.github/workflows/test.yml +++ b/.github/workflows/test.yml @@ -39,6 +39,15 @@ jobs: - name: Check Formatting run: cargo +nightly fmt --all -- --check + + - name: Check Stable Compilation + run: cargo build --all-features + + - name: Check Nightly Compilation + run: cargo +nightly build --all-features + + - name: Check Bench Compilation + run: cargo +nightly bench --no-run --profile=dev --all-features - uses: actions-rs/clippy-check@v1 with: From c588b1d325091feff4a8c10b87143dc1624ace64 Mon Sep 17 00:00:00 2001 From: PSeitz Date: Thu, 12 Oct 2023 12:22:22 +0200 Subject: [PATCH 2/2] Apply suggestions from code review Co-authored-by: Paul Masurel --- .github/workflows/test.yml | 2 -- 1 file changed, 2 deletions(-) diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml index 64d413acbf..13080f11d2 100644 --- a/.github/workflows/test.yml +++ b/.github/workflows/test.yml @@ -43,8 +43,6 @@ jobs: - name: Check Stable Compilation run: cargo build --all-features - - name: Check Nightly Compilation - run: cargo +nightly build --all-features - name: Check Bench Compilation run: cargo +nightly bench --no-run --profile=dev --all-features