From 0b644170586c94bf7fd19ab384255b4556ae4922 Mon Sep 17 00:00:00 2001 From: Manuel Raynaud Date: Mon, 1 Sep 2025 14:29:47 +0200 Subject: [PATCH] =?UTF-8?q?=F0=9F=94=A7(ci)=20always=20run=20all=20git-lin?= =?UTF-8?q?t=20steps=20(#1323)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit git-lint steps are independant and we would like to have all checks at once. Using the `if: always()` instruction should ensure all steps should be run event if the previous fails. --- .github/workflows/impress.yml | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/.github/workflows/impress.yml b/.github/workflows/impress.yml index 482550d0..8cc2ff82 100644 --- a/.github/workflows/impress.yml +++ b/.github/workflows/impress.yml @@ -25,14 +25,18 @@ jobs: - name: show run: git log - name: Enforce absence of print statements in code + if: always() run: | ! git diff origin/${{ github.event.pull_request.base.ref }}..HEAD -- . ':(exclude)**/impress.yml' | grep "print(" - name: Check absence of fixup commits + if: always() run: | ! git log | grep 'fixup!' - name: Install gitlint + if: always() run: pip install --user requests gitlint - name: Lint commit messages added to main + if: always() run: ~/.local/bin/gitlint --commits origin/${{ github.event.pull_request.base.ref }}..HEAD check-changelog: