SRE-1110: Create pull request template - #113
TimDiekmann wants to merge 1 commit into
Conversation
PR SummaryLow Risk Overview The template defines four sections—Purpose, Changes, Known issues, and Blocked by—each with HTML comment guidance on what to write, when a section is required vs. optional, and reminders (e.g. human-written purpose, no internal/security details in descriptions, omit obvious stacked-PR dependencies). Reviewed by Cursor Bugbot for commit eb5c464. Bugbot is set up for automated code reviews on this repo. Configure here. |
| <!-- | ||
| Describe anything that must be done before the PR is merged, for example further testing, deployment of something, etc. Delete this section otherwise. | ||
| -> Never include dependencies that are already obvious in GitHub (stacked branches). | ||
| --> |
There was a problem hiding this comment.
I did like the "how to test this"/"Demo" section. Can we add it and make it commented out by default perhaps?
The Linear issue talks about this potentially replacing the other templates in the main repo, is that something you expect?
There was a problem hiding this comment.
Could you raise this in this slack thread? We have the context there already.
🌟 Purpose
We currently only have a single PR template in our main repo. This introduces a global one.