From c5d0b3cf84d9c308e7d56b1a0dadb44470962e41 Mon Sep 17 00:00:00 2001 From: Antoine du Hamel Date: Fri, 12 Nov 2021 23:35:22 +0100 Subject: [PATCH] tools: only validate first commit message of a PR MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit PR-URL: https://github.com/nodejs/node/pull/40740 Reviewed-By: Michaël Zasso Reviewed-By: Luigi Pinca --- .github/workflows/commit-lint.yml | 11 ++++------- 1 file changed, 4 insertions(+), 7 deletions(-) diff --git a/.github/workflows/commit-lint.yml b/.github/workflows/commit-lint.yml index f362f6103f7b1f..1609d239404241 100644 --- a/.github/workflows/commit-lint.yml +++ b/.github/workflows/commit-lint.yml @@ -1,4 +1,4 @@ -name: "Commit messages adheres to guidelines at https://goo.gl/p2fr5Q" +name: "First commit message adheres to guidelines at https://goo.gl/p2fr5Q" on: [pull_request] @@ -9,17 +9,14 @@ jobs: lint-commit-message: runs-on: ubuntu-latest steps: - - name: Compute number of commits in the PR - id: nb-of-commits - run: echo "::set-output name=nb::$((${{ github.event.pull_request.commits }} + 1))" - uses: actions/checkout@v2 with: - fetch-depth: ${{ steps.nb-of-commits.outputs.nb }} + fetch-depth: 2 - name: Install Node.js uses: actions/setup-node@v2 with: node-version: ${{ env.NODE_VERSION }} - - name: Validate commit messages + - name: Validate commit message run: | echo "::add-matcher::.github/workflows/commit-lint-problem-matcher.json" - git log --oneline -${{ github.event.pull_request.commits }} HEAD^ | awk '{ if ($2 != "fixup!" && $2 != "squash!") { print $1 } }' | xargs npx -q core-validate-commit --no-validate-metadata --tap + git rev-parse HEAD^ | xargs npx -q core-validate-commit --no-validate-metadata --tap