Skip to content

Commit c464f93

Browse files
imaqsoodclaude
andcommitted
(MODULES-11832) Add synced PR template with outcome-based review policy
Add moduleroot/.github/PULL_REQUEST_TEMPLATE.md.erb so every CAT-supported module gets a synced PR template on `pdk update`. The template enforces the S2 outcome-based review policy: - Required checklist: structured-ticket linkage, coverage gate green/baseline preserved, platform matrix result link, Co-Authored-By: Claude trailer when AI-assisted. - Outcome-based review lanes (auto-merge / single reviewer / full review), consistent with the policy documented in moduleroot/CLAUDE.md. Managed (not unmanaged) so the policy stays consistent across modules; a module can opt out via `.sync.yml`. Verified with `pdk convert --noop` on puppetlabs-lvm: renders to .github/PULL_REQUEST_TEMPLATE.md. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
1 parent cd0953f commit c464f93

1 file changed

Lines changed: 50 additions & 0 deletions

File tree

Lines changed: 50 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,50 @@
1+
<!--
2+
This pull request template is synced from puppetlabs/pdk-templates
3+
(moduleroot/.github/PULL_REQUEST_TEMPLATE.md.erb). Do not edit it directly in a
4+
module repo -- changes are overwritten on the next `pdk update`. To customise it
5+
for a single module, opt out with `.github/PULL_REQUEST_TEMPLATE.md: unmanaged: true`
6+
in the module's `.sync.yml` and maintain a hand-written copy.
7+
-->
8+
9+
## Summary
10+
11+
Describe what this change does and why. Keep the change scoped to the linked ticket;
12+
flag any unrelated cleanup as a separate ticket/PR.
13+
14+
## Related ticket
15+
16+
<!-- Link the driving ticket using the structured spec template. -->
17+
- Ticket: MODULES-XXXX
18+
19+
## Required checklist
20+
21+
- [ ] Linked the driving `MODULES-*` ticket above using the structured spec template.
22+
- [ ] Coverage gate is green (or the baseline is preserved -- coverage was not reduced).
23+
- [ ] Platform matrix result link included below (CI run or nightly Litmus result).
24+
- [ ] `Co-Authored-By: Claude` trailer is present on the commits if this PR was AI-assisted.
25+
26+
## Platform matrix result
27+
28+
<!-- Paste the link to the CI / nightly Litmus run that proves the supported OS matrix is green. -->
29+
30+
## Review policy (outcome-based)
31+
32+
PRs are judged on outcomes, not line counts. Pick the lane that matches this change so
33+
reviewers know what coverage to expect. See `CLAUDE.md` for the full policy.
34+
35+
- **Auto-merge** -- green CI required, no human review needed:
36+
dependabot bumps, modulesync / `pdk update` resync, doc-only changes, `REFERENCE.md` regeneration.
37+
- **Single reviewer** -- green CI plus coverage preserved required:
38+
bug fixes, supported-platform additions, simple feature work.
39+
- **Full review (2 or more reviewers)** -- green CI plus coverage plus senior sign-off:
40+
provider / type-API changes, security fixes, breaking changes.
41+
42+
This PR is (check one):
43+
44+
- [ ] Auto-merge
45+
- [ ] Single reviewer
46+
- [ ] Full review (2+)
47+
48+
## Additional context
49+
50+
<!-- Root cause, reproduction steps, and the reasoning behind the implementation, if applicable. -->

0 commit comments

Comments
 (0)