chore: smoke test post-merge celebration workflow#1038
Conversation
HTML comment only at README foot—no rendered change. Made-with: Cursor
|
🎉 Merged — thanks @muddlebee! Your change is in; appreciate you taking the time to contribute. 💬 Community: Discord — OpenSRE ( |
Greptile SummaryThis PR adds a single invisible HTML comment ( Confidence Score: 5/5Safe to merge — only an invisible HTML comment is added; no functional code is changed. The sole change is a non-rendering HTML comment in README.md. The workflow and Python script it exercises are already on the base branch and contain no logic issues or security concerns (pull_request_target checks out the base branch only, contributor login is a validated GitHub username, no secrets are exposed). All findings are P2 or lower — score is 5. No files require special attention. Important Files Changed
Sequence DiagramsequenceDiagram
participant Dev as Contributor (muddlebee)
participant GH as GitHub (PR merge)
participant WF as celebrate-merged-pr.yml
participant PY as merged_pr_celebration_message.py
participant GH_API as GitHub API (gh cli)
Dev->>GH: Merge PR #1038 (README.md HTML comment)
GH->>WF: pull_request_target [closed] event
WF->>WF: Check merged==true && user.type!=Bot
WF->>WF: actions/checkout@v5 (base branch only)
WF->>PY: python3 .github/scripts/merged_pr_celebration_message.py
PY->>PY: Read CONTRIBUTOR_LOGIN + DISCORD_INVITE_URL
PY->>PY: Pick random template + optional GIF
PY-->>WF: Writes comment.md
WF->>GH_API: gh pr comment #1038 --body-file comment.md
GH_API-->>GH: Post celebration comment on PR
Reviews (1): Last reviewed commit: "chore: smoke test post-merge celebration..." | Re-trigger Greptile |

Smoke PR to verify
.github/workflows/celebrate-merged-pr.ymlposts a congratulatory Discord comment after merge.After merge: expect a workflow comment on this PR with
@mention+ Discord link.Change: HTML comment only at end of
README.md(no visible README diff).Made with Cursor