Skip to content

Commit 09e3590

Browse files
aihazmAiham Azmeh
andauthored
fix: pr comment ci job previously flooding the PR comments (#15)
Co-authored-by: Aiham Azmeh <aiham.azmeh@gmail.com>
1 parent 857e54a commit 09e3590

File tree

1 file changed

+10
-10
lines changed

1 file changed

+10
-10
lines changed

.github/workflows/comment_checklist_pr.yml

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -3,10 +3,10 @@ name: Comment on PR
33
on:
44
pull_request:
55
types: [opened, synchronize]
6-
branches: [ "main" ]
6+
branches: ["main"]
77

88
workflow_dispatch:
9-
9+
1010
jobs:
1111
verify:
1212
runs-on: ubuntu-latest
@@ -15,38 +15,38 @@ jobs:
1515
steps:
1616
- name: Checkout
1717
uses: actions/checkout@v6
18-
18+
1919
- name: Retrieve app name from release config
2020
run: |
2121
APPNAME=$(jq -r '.name' "release.json")
22-
echo "APPNAME=$APPNAME" >> $GITHUB_ENV
23-
22+
echo "APPNAME=$APPNAME" >> $GITHUB_ENV
23+
2424
- name: PR comment with file
2525
uses: thollander/actions-comment-pull-request@v3
2626
with:
2727
mode: upsert
28-
comment_tag: previewComment
28+
comment-tag: previewComment
2929
message: |
3030
# PR checklist
31-
31+
3232
Checklist:
33-
33+
3434
- [ ] Is the name of this app/solution still `${{ env.APPNAME }}`?
3535
- [ ] Have you updated the version tag in each .qvf file, and followed semantic versioning (e.g. v1.1.1)?
3636
- [ ] Is this PR named appropriately? For example:
3737
- Feature release: `feat: add sheet usage event`
3838
- Bug fix: `fix: replaced deprecated events`
3939
- Documentation update: `docs: updated images in install guide`
4040
- [ ] Have you reviewed the unbuilt diff generated by the workflow in /diff?
41-
41+
4242
If you wish to merge and release:
4343
4444
- Have this PR approved by a code owner, then merge into main
4545
- Go to releases, an action will generate a draft release with all files in /assets, and any external assets references in `release.json`
4646
- Make any updates, then submit the release
4747
4848
Common issues:
49-
49+
5050
- After merging, draft release doesn't update - this is usually because
5151
the action failed due to an old draft release with the same name.
5252
Delete the draft release, and re-run the action.

0 commit comments

Comments
 (0)