From ffbb3010149f73b254916657f2ce847865332dbb Mon Sep 17 00:00:00 2001 From: Shahar Naveh <50263213+ShaharNaveh@users.noreply.github.com> Date: Fri, 10 Apr 2026 07:30:31 +0200 Subject: [PATCH] Revert "Only run `cargo check` when rust code is changed (#7572)" This reverts commit ad66d9acd0f3fca9111f6ea18a719c8b540ae76f. --- .github/workflows/ci.yaml | 42 +-------------------------------------- 1 file changed, 1 insertion(+), 41 deletions(-) diff --git a/.github/workflows/ci.yaml b/.github/workflows/ci.yaml index 0498398196..bb78a825a4 100644 --- a/.github/workflows/ci.yaml +++ b/.github/workflows/ci.yaml @@ -32,42 +32,6 @@ env: CARGO_TERM_COLOR: always jobs: - determine_changes: - name: Determine changes - runs-on: ubuntu-latest - outputs: - # Flag that is raised when any rust code is changed. - rust_code: ${{ steps.check_rust_code.outputs.changed }} - steps: - - uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2 - with: - fetch-depth: 0 - persist-credentials: false - - - name: Determine merge base - id: merge_base - run: | - sha=$(git merge-base HEAD "origin/${BASE_REF}") - echo "sha=${sha}" >> "$GITHUB_OUTPUT" - env: - BASE_REF: ${{ github.event.pull_request.base.ref || 'main' }} - - - name: Check if there was any code related change - id: check_rust_code - run: | - if git diff --quiet "${MERGE_BASE}...HEAD" -- \ - ':!Lib/**' \ - ':!scripts/**' \ - ':!extra_tests/**' \ - ':.github/workflows/ci.yaml' \ - ; then - echo "changed=false" >> "$GITHUB_OUTPUT" - else - echo "changed=true" >> "$GITHUB_OUTPUT" - fi - env: - MERGE_BASE: ${{ steps.merge_base.outputs.sha }} - rust_tests: if: ${{ !contains(github.event.pull_request.labels.*.name, 'skip:ci') }} env: @@ -145,13 +109,9 @@ jobs: if: runner.os == 'Linux' cargo_check: + if: ${{ !contains(github.event.pull_request.labels.*.name, 'skip:ci') }} name: cargo check runs-on: ${{ matrix.os }} - needs: - - determine_changes - if: | - !contains(github.event.pull_request.labels.*.name, 'skip:ci') && - needs.determine_changes.outputs.rust_code == 'true' strategy: matrix: include: