diff options
| author | Craig Jennings <c@cjennings.net> | 2026-05-01 14:34:49 -0500 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-05-01 14:34:49 -0500 |
| commit | b5a5b9fb5f8f0abdb87553882ce26cd2ad5792bc (patch) | |
| tree | 0bb3281b0fd4bf316e874bf80433d7f29f49b1ea /debug | |
| parent | 5dc03427571d88d407389e67d1c1c1936e8f61ce (diff) | |
| download | rulesets-b5a5b9fb5f8f0abdb87553882ce26cd2ad5792bc.tar.gz rulesets-b5a5b9fb5f8f0abdb87553882ce26cd2ad5792bc.zip | |
docs(commits): add post-review Slack notify + approve-vs-merge note
Step 8 of the PR review/publish flow now sends a one-line message to channel C0AM2MWHCJU after every approve or changes-requested verdict. The new step skips humanizer and personal-style passes since the message is short and templated.
The approve case sends "Approved on PR #N." with the PR URL on the next line. The changes-requested case sends "Changes Requested on PR #N" with the PR URL on the next line.
I also added an explicit line saying approve doesn't authorize a merge. That keeps the merge decision with the PR author. The team's practice is approve-then-author-merges, not approve-and-merge.
Diffstat (limited to 'debug')
0 files changed, 0 insertions, 0 deletions