diff --git a/.github/workflows/coveralls.yml b/.github/workflows/coveralls.yml index 25d98c28..a3d37cd3 100644 --- a/.github/workflows/coveralls.yml +++ b/.github/workflows/coveralls.yml @@ -5,20 +5,7 @@ on: name: Code coverage with llvm-cov jobs: - Prepare: - runs-on: ubuntu-latest - outputs: - pr_changed_source: ${{ steps.classify_pr.outputs.pr_changed_source }} - steps: - - name: Checkout Crate - uses: actions/checkout@v4 - - name: Determine what files the PR changes. - id: classify_pr - run: contrib/classify-pr.sh ${{ github.event.pull_request.head.sha }} ${{ github.event.pull_request.base.sha }} - Coveralls: - needs: Prepare - if: needs.Prepare.outputs.pr_changed_source == 'true' name: Code coverage - stable toolchain runs-on: ubuntu-latest strategy: diff --git a/.github/workflows/fuzz.yml b/.github/workflows/fuzz.yml index 232d8812..3a0e4d1a 100644 --- a/.github/workflows/fuzz.yml +++ b/.github/workflows/fuzz.yml @@ -9,20 +9,8 @@ on: pull_request: jobs: - Prepare: - runs-on: ubuntu-latest - outputs: - pr_changed_source: ${{ steps.classify_pr.outputs.pr_changed_source }} - steps: - - name: Checkout Crate - uses: actions/checkout@v4 - - name: Determine what files the PR changes. - id: classify_pr - run: contrib/classify-pr.sh ${{ github.event.pull_request.head.sha }} ${{ github.event.pull_request.base.sha }} - fuzz: - needs: Prepare - if: ${{ !github.event.act }} && needs.Prepare.outputs.pr_changed_source == 'true' + if: ${{ !github.event.act }} runs-on: ubuntu-latest strategy: fail-fast: false @@ -76,8 +64,7 @@ jobs: path: executed_${{ matrix.fuzz_target }} verify-execution: - needs: Prepare - if: ${{ !github.event.act }} && needs.Prepare.outputs.pr_changed_source == 'true' + if: ${{ !github.event.act }} needs: fuzz runs-on: ubuntu-latest steps: diff --git a/.github/workflows/rust.yml b/.github/workflows/rust.yml index 6081d431..169e2e5f 100644 --- a/.github/workflows/rust.yml +++ b/.github/workflows/rust.yml @@ -14,13 +14,9 @@ jobs: crates: ${{ steps.get_matrix.outputs.crates }} deps: ${{ steps.get_matrix.outputs.deps }} nightly_version: ${{ steps.read_toolchain.outputs.nightly_version }} - pr_changed_source: ${{ steps.classify_pr.outputs.pr_changed_source }} steps: - name: Checkout Crate uses: actions/checkout@v4 - - name: Determine what files the PR changes. - id: classify_pr - run: contrib/classify-pr.sh ${{ github.event.pull_request.head.sha }} ${{ github.event.pull_request.base.sha }} - name: Read nightly version id: read_toolchain run: echo "nightly_version=$(cat .github/nightly-version)" >> $GITHUB_OUTPUT @@ -30,7 +26,6 @@ jobs: Stable: needs: Prepare - if: needs.Prepare.outputs.pr_changed_source == 'true' name: Test - stable toolchain runs-on: ubuntu-latest strategy: @@ -52,7 +47,6 @@ jobs: Beta: needs: Prepare - if: needs.Prepare.outputs.pr_changed_source == 'true' name: Test - beta toolchain runs-on: ubuntu-latest strategy: @@ -73,7 +67,6 @@ jobs: Nightly: needs: Prepare - if: needs.Prepare.outputs.pr_changed_source == 'true' name: Test - nightly toolchain runs-on: ubuntu-latest strategy: @@ -98,7 +91,6 @@ jobs: MSRV: needs: Prepare - if: needs.Prepare.outputs.pr_changed_source == 'true' name: Test - 1.56.1 toolchain runs-on: ubuntu-latest strategy: @@ -120,8 +112,6 @@ jobs: run: ./contrib/run_task.sh ${{ matrix.crate }} ${{ matrix.task }} Arch32bit: - needs: Prepare - if: needs.Prepare.outputs.pr_changed_source == 'true' name: Test 32-bit version runs-on: ubuntu-latest steps: @@ -139,9 +129,8 @@ jobs: run: cargo test --target i686-unknown-linux-gnu Cross: - needs: Prepare - if: ${{ !github.event.act }} && needs.Prepare.outputs.pr_changed_source == 'true' name: Cross test + if: ${{ !github.event.act }} runs-on: ubuntu-latest steps: - name: Checkout Crate @@ -157,7 +146,6 @@ jobs: Embedded: needs: Prepare - if: needs.Prepare.outputs.pr_changed_source == 'true' runs-on: ubuntu-latest env: RUSTFLAGS: "-C link-arg=-Tlink.x" @@ -183,7 +171,6 @@ jobs: ASAN: needs: Prepare - if: needs.Prepare.outputs.pr_changed_source == 'true' name: Address sanitizer # hashes crate only. runs-on: ubuntu-latest strategy: @@ -206,7 +193,6 @@ jobs: WASM: needs: Prepare - if: needs.Prepare.outputs.pr_changed_source == 'true' name: WebAssembly Build # hashes crate only. runs-on: ubuntu-latest strategy: @@ -225,7 +211,6 @@ jobs: Schemars: needs: Prepare - if: needs.Prepare.outputs.pr_changed_source == 'true' name: Schemars runs-on: ubuntu-latest strategy: @@ -243,8 +228,6 @@ jobs: run: ./contrib/run_task.sh ${{ matrix.crate }} ${{ matrix.task }} Kani: - needs: Prepare - if: needs.Prepare.outputs.pr_changed_source == 'true' runs-on: ubuntu-20.04 steps: - name: 'Checkout your code.' diff --git a/contrib/classify-pr.sh b/contrib/classify-pr.sh deleted file mode 100755 index e72fba5d..00000000 --- a/contrib/classify-pr.sh +++ /dev/null @@ -1,29 +0,0 @@ -#!/bin/sh - -if [ "$#" -ne 2 ]; then - echo "Usage: $0 " - exit 1 -fi - -pr_tip=$1 -master_tip=$2 - -# When called on `pull_request`, GA fails to pull down master by default. -# When called on `push`, GA fails to pull down the PR by default, only its merge commit. -# The simplest way to deal with this is to just pull both explicitly. -git fetch origin "$master_tip":master_tip -git fetch origin "$pr_tip":pr_tip - -pr_base=$(git merge-base master_tip pr_tip) - -echo "Using master $master_tip" -echo "Using PR tip $pr_tip" -echo "Using PR base $pr_base" - -# If something modifies any non-markdown file, it's considered a source code change. -if git diff --name-only "$pr_base" "$pr_tip" | grep -qv "^.md$"; then - echo "pr_changed_source=true" >> "$GITHUB_OUTPUT" -else - echo "pr_changed_source=false" >> "$GITHUB_OUTPUT" -fi -