diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml index 024d18bd0fc..7ce22befce7 100644 --- a/.github/workflows/tests.yml +++ b/.github/workflows/tests.yml @@ -282,10 +282,26 @@ jobs: - check-schema-delta - check-lockfile - lint-clippy + - lint-clippy-nightly - lint-rustfmt runs-on: ubuntu-latest steps: - - run: "true" + - uses: matrix-org/done-action@v2 + with: + needs: ${{ toJSON(needs) }} + + # Various bits are skipped if there was no applicable changes. + skippable: | + check-sampleconfig + check-schema-delta + lint + lint-mypy + lint-newsfile + lint-pydantic + lint-clippy + lint-clippy-nightly + lint-rustfmt + calculate-test-jobs: if: ${{ !cancelled() && !failure() }} # Allow previous steps to be skipped, but not fail @@ -695,6 +711,7 @@ jobs: - complement - cargo-test - cargo-bench + - linting-done runs-on: ubuntu-latest steps: - uses: matrix-org/done-action@v2 diff --git a/changelog.d/16781.misc b/changelog.d/16781.misc new file mode 100644 index 00000000000..2f628dc5cbd --- /dev/null +++ b/changelog.d/16781.misc @@ -0,0 +1 @@ +Ensure CI fails when linting fails to make sure auto-merge does the correct thing.