From afda8f7846dec564b890888f7c863c3e03d73baa Mon Sep 17 00:00:00 2001 From: Luke Alvoeiro Date: Wed, 4 Sep 2024 10:32:37 -0700 Subject: [PATCH] PR comment to indicate cause of failure --- .github/workflows/pull_request_title.yaml | 31 +++++++++++++++++++++-- 1 file changed, 29 insertions(+), 2 deletions(-) diff --git a/.github/workflows/pull_request_title.yaml b/.github/workflows/pull_request_title.yaml index b06f470a..8a32172a 100644 --- a/.github/workflows/pull_request_title.yaml +++ b/.github/workflows/pull_request_title.yaml @@ -1,4 +1,4 @@ -name: "Lint PR" +name: 'Lint PR' on: pull_request_target: @@ -9,7 +9,7 @@ on: - reopened permissions: - pull-requests: read + pull-requests: write jobs: main: @@ -17,5 +17,32 @@ jobs: runs-on: ubuntu-latest steps: - uses: amannn/action-semantic-pull-request@v5 + id: lint_pr_title env: GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} + with: + requireScope: false + + - uses: marocchino/sticky-pull-request-comment@v2 + # When the previous steps fails, the workflow would stop. By adding this + # condition you can continue the execution with the populated error message. + if: always() && (steps.lint_pr_title.outputs.error_message != null) + with: + header: pr-title-lint-error + message: | + Hey there and thank you for opening this pull request! 👋🏼 + + We require pull request titles to follow the [Conventional Commits specification](https://gist.github.com/Zekfad/f51cb06ac76e2457f11c80ed705c95a3#file-conventional-commits-md) and it looks like your proposed title needs to be adjusted. + + Details: + + ``` + ${{ steps.lint_pr_title.outputs.error_message }} + ``` + + # Delete a previous comment when the issue has been resolved + - if: ${{ steps.lint_pr_title.outputs.error_message == null }} + uses: marocchino/sticky-pull-request-comment@v2 + with: + header: pr-title-lint-error + delete: true