From 34694997ecb81751c542d8618edaee770a6a3d5a Mon Sep 17 00:00:00 2001 From: "J. Nick Koston" Date: Sat, 12 Jul 2025 14:09:02 -1000 Subject: [PATCH] Fix clang-tidy skipping when Python linters are skipped --- .github/workflows/ci.yml | 27 +++++++++++++++++++-------- 1 file changed, 19 insertions(+), 8 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 503a50c5c2..07d38a286c 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -345,18 +345,28 @@ jobs: run: script/ci-suggest-changes if: always() + clang-tidy-deps: + name: Clang-tidy dependencies + runs-on: ubuntu-24.04 + needs: + - common + - ci-custom + - clang-format + - pytest + - determine-jobs + - ruff + if: | + always() && + needs.determine-jobs.outputs.clang-tidy == 'true' && + (needs.ruff.result == 'success' || needs.ruff.result == 'skipped') + steps: + - run: echo "All clang-tidy dependencies ready" + clang-tidy: name: ${{ matrix.name }} runs-on: ubuntu-24.04 needs: - - common - - ruff - - ci-custom - - clang-format - - flake8 - - pylint - - pytest - - pyupgrade + - clang-tidy-deps - determine-jobs if: needs.determine-jobs.outputs.clang-tidy == 'true' env: @@ -575,6 +585,7 @@ jobs: - pytest - integration-tests - pyupgrade + - clang-tidy-deps - clang-tidy - determine-jobs - test-build-components