refactor: apply suggestions

This commit is contained in:
Jordan Labrosse 2025-09-16 09:47:26 +02:00
parent e76373cff7
commit 756a484c0f
No known key found for this signature in database

View file

@ -1,8 +1,11 @@
name: Conventional Commits Check name: Conventional Commits Check
on: on:
pull_request: pull_request_target:
types: [opened, synchronize, reopened, edited] types:
- opened
- edited
- reopened
jobs: jobs:
check-conventional-commits: check-conventional-commits:
@ -11,39 +14,8 @@ jobs:
permissions: permissions:
pull-requests: read pull-requests: read
steps: steps:
- uses: actions/checkout@v5
with:
fetch-depth: 0
- name: Check Commit Conventions
uses: webiny/action-conventional-commits@v1.3.0
- name: Check Semantic Pull Request title - name: Check Semantic Pull Request title
uses: amannn/action-semantic-pull-request@v6 uses: amannn/action-semantic-pull-request@v6
id: lint_pr_title id: lint_pr_title
env: env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
- uses: marocchino/sticky-pull-request-comment@v2
# When the previous steps fail, 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://www.conventionalcommits.org/en/v1.0.0/) 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