From 637d66e9b44faaddceaf7dcc7e0f8b752092189a Mon Sep 17 00:00:00 2001 From: Erik Johnston Date: Fri, 5 Jan 2024 13:19:59 +0000 Subject: [PATCH 1/2] Wait on linting-done --- .github/workflows/tests.yml | 19 ++++++++++++++++++- 1 file changed, 18 insertions(+), 1 deletion(-) diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml index f22ca5f7e60..99d6c1bceb0 100644 --- a/.github/workflows/tests.yml +++ b/.github/workflows/tests.yml @@ -286,10 +286,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 @@ -699,6 +715,7 @@ jobs: - complement - cargo-test - cargo-bench + - linting-done runs-on: ubuntu-latest steps: - uses: matrix-org/done-action@v2 From 42fead6c5623e6d747a47ed72187a51f87950087 Mon Sep 17 00:00:00 2001 From: Erik Johnston Date: Fri, 5 Jan 2024 13:28:34 +0000 Subject: [PATCH 2/2] Newsfile --- changelog.d/16781.misc | 1 + 1 file changed, 1 insertion(+) create mode 100644 changelog.d/16781.misc 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.