Skip to content

Commit f5b7a7e

Browse files
ashleyshawashleyshaw
committed
2 parents 0a179df + 2fad6c1 commit f5b7a7e

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

55 files changed

+8431
-1407
lines changed

‎.github/PULL_REQUEST_TEMPLATE/pr_bug.md‎

Lines changed: 63 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -1,46 +1,94 @@
11
---
2-
name: "Bugfix PR"
3-
about: "Fix a defect/regression with clear repro, fix, and verification"
4-
title: "fix: <short summary>"
5-
labels: ["status:needs-review"]
2+
3+
name: "Bugfix PR"
4+
about: "Fix a defect/regression with clear repro, fix, and verification"
5+
title: "fix: {short summary}"
6+
labels: ["status:needs-review"]
7+
68
---
7-
<!-- Note: YAML front matter is parsed by our labeler workflow; GitHub itself does not apply labels from front matter. -->
89

9-
<!-- Suggested additional labels (pick as relevant):
10-
- priority:* (set by impact)
11-
- env:staging (for QA)
10+
# Bugfix Pull Request
11+
12+
> This repository enforces changelog, release, and label automation for all PRs and issues.
13+
> See the organisation-wide [Automation Governance & Release Strategy](https://github.com/lightspeedwp/.github/blob/main/AUTOMATION_GOVERNANCE.md) for required rules.
14+
15+
## Linked issues
16+
17+
<!--
18+
List any related issues by number (e.g. fixes #123, closes #456, relates to #789).
1219
-->
1320

21+
Fixes #
22+
1423
## Context
15-
- Severity/Impact: <High/Medium/Low>
16-
- Affected versions/environments: <list>
24+
25+
- Severity/Impact: (High/Medium/Low)
26+
- Affected versions/environments: (list)
1727

1828
## Reproduction
29+
1930
- Steps: 1) … 2) … 3) …
20-
- Expected vs Actual: <summary>
31+
- Expected vs Actual: (summary)
2132

2233
## Root Cause
23-
- <brief analysis and evidence (logs/links)>
34+
35+
- (brief analysis and evidence (logs/links))
2436

2537
## Fix Summary
26-
- <what changed and why>
38+
39+
- (what changed and why)
2740

2841
## Verification
42+
2943
- [ ] Tests added/updated to cover the bug
3044
- [ ] Manual verification steps (browsers/devices)
3145
- [ ] Negative/edge cases checked
3246

3347
## Risk & Rollback
48+
3449
- Risk level: Low / Medium / High
35-
- Rollback plan: <revert / feature-flag / config>
50+
- Rollback plan: (revert / feature-flag / config)
51+
52+
## Changelog
53+
54+
<!--
55+
Required for release automation.
56+
Format: Keep a Changelog.
57+
Categories: Added, Changed, Fixed, Removed.
58+
User-facing notes only. Internal-only PRs (rare) may use the skip-changelog label.
59+
Example:
60+
### Fixed
61+
- Correct VAT rounding on order totals in EU regions. (Fixes #456, @alice)
62+
-->
63+
64+
### Added
65+
66+
- [placeholder]
67+
68+
### Changed
69+
70+
- [placeholder]
71+
72+
### Fixed
73+
74+
- [placeholder]
75+
76+
### Removed
77+
78+
- [placeholder]
79+
80+
<!--
81+
If no user-facing changelog entry is needed, apply the skip-changelog label to this PR.
82+
-->
3683

3784
---
85+
3886
### Checklist (Global DoD / PR)
87+
3988
- [ ] All AC met and demonstrated
4089
- [ ] Tests added/updated (unit/E2E as appropriate)
4190
- [ ] A11y considerations addressed where relevant
4291
- [ ] Docs/readme/changelog updated (if user-facing)
4392
- [ ] Security/perf impact reviewed where relevant
4493
- [ ] Code/design reviews approved
4594
- [ ] CI green; linked issues closed; release notes prepared (if shipping)
46-

‎.github/PULL_REQUEST_TEMPLATE/pr_chore.md‎

Lines changed: 56 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -1,27 +1,36 @@
11
---
2-
name: "Chore PR"
3-
about: "Repo hygiene: configs, scripts, formatting — no behaviour change"
4-
title: "chore: <scope>"
5-
labels: ["status:needs-review"]
2+
3+
name: "Chore PR"
4+
about: "Repo hygiene: configs, scripts, formatting — no behaviour change"
5+
title: "chore: {scope}"
6+
labels: ["status:needs-review"]
7+
68
---
7-
<!-- Note: YAML front matter is parsed by our labeler workflow; GitHub itself does not apply labels from front matter. -->
89

9-
<!-- Suggested additional labels (pick as relevant):
10-
- area:dependencies or area:ci (if relevant)
10+
# Chore Pull Request
11+
12+
> This repository enforces changelog, release, and label automation for all PRs and issues.
13+
> See the organisation-wide [Automation Governance & Release Strategy](https://github.com/lightspeedwp/.github/blob/main/AUTOMATION_GOVERNANCE.md) for required rules.
14+
15+
16+
## Linked issues
17+
18+
<!--
19+
List any related issues by number (e.g. closes #123, relates to #789).
1120
-->
1221

13-
## Summary
22+
Closes #
1423

15-
**Linked issues**: Closes #<id> (and/or) Relates to #<id>
24+
## Summary
1625

1726
## Changes
1827

19-
- <list>
28+
- (list)
2029

2130
## Impact / Compatibility
2231

23-
- Runtime/behaviour changes: <None expected>
24-
- Build/dev-experience impact: <notes>
32+
- Runtime/behaviour changes: (None expected)
33+
- Build/dev-experience impact: (notes)
2534

2635
## Verification
2736

@@ -32,16 +41,49 @@ labels: ["status:needs-review"]
3241
## Risk & Rollback
3342

3443
- Risk level: Low / Medium / High
35-
- Rollback plan: <revert commit>
44+
- Rollback plan: (revert commit)
45+
46+
## Changelog
47+
48+
<!--
49+
Required for release automation.
50+
Format: Keep a Changelog.
51+
Categories: Added, Changed, Fixed, Removed.
52+
User-facing notes only. Internal-only PRs (rare) may use the skip-changelog label.
53+
Example:
54+
### Changed
55+
- Updated CI config for improved cache usage. (Relates to #789)
56+
-->
57+
58+
### Added
59+
60+
- [placeholder]
61+
62+
### Changed
63+
64+
- [placeholder]
65+
66+
### Fixed
67+
68+
- [placeholder]
69+
70+
### Removed
71+
72+
- [placeholder]
73+
74+
<!--
75+
If no user-facing changelog entry is needed, apply the skip-changelog label to this PR.
76+
-->
3677

3778
---
3879

80+
3981
### Checklist (Global DoD / PR)
82+
4083
- [ ] All AC met and demonstrated
4184
- [ ] Tests added/updated (unit/E2E as appropriate)
4285
- [ ] A11y considerations addressed where relevant
4386
- [ ] Docs/readme/changelog updated (if user-facing)
4487
- [ ] Security/perf impact reviewed where relevant
4588
- [ ] Code/design reviews approved
4689
- [ ] CI green; linked issues closed; release notes prepared (if shipping)
47-

‎.github/PULL_REQUEST_TEMPLATE/pr_ci.md‎

Lines changed: 55 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1,19 +1,29 @@
11
---
2-
name: "Build/CI PR"
3-
about: "Pipelines, linting, packaging, or release automation"
4-
title: "build(ci): <short summary>"
5-
labels: ["status:needs-review", "area:ci"]
2+
3+
name: "Build/CI PR"
4+
about: "Pipelines, linting, packaging, or release automation"
5+
title: "build(ci): {short summary}"
6+
labels: ["status:needs-review", "area:ci"]
7+
68
---
7-
<!-- Note: YAML front matter is parsed by our labeler workflow; GitHub itself does not apply labels from front matter. -->
89

9-
<!-- Suggested additional labels (pick as relevant):
10-
- compat:wordpress (if build targets)
10+
# Build/CI Pull Request
11+
12+
> This repository enforces changelog, release, and label automation for all PRs and issues.
13+
> See the organisation-wide [Automation Governance & Release Strategy](https://github.com/lightspeedwp/.github/blob/main/AUTOMATION_GOVERNANCE.md) for required rules.
14+
15+
## Linked issues
16+
17+
<!--
18+
List any related issues by number (e.g. closes #123, relates to #789).
1119
-->
1220

21+
Relates to #
22+
1323
## Build/CI change
1424

15-
- What: <summarise>
16-
- Why: <reliability/speed/consistency>
25+
- What: (summarise)
26+
- Why: (reliability/speed/consistency)
1727

1828
## Baseline & Target
1929

@@ -22,20 +32,53 @@ labels: ["status:needs-review", "area:ci"]
2232

2333
## Rollback
2434

25-
- Plan: <how to revert>
35+
- Plan: (how to revert)
2636

2737
## Notes
2838

29-
- Secrets/permissions considerations: <details>
39+
- Secrets/permissions considerations: (details)
40+
41+
## Changelog
42+
43+
<!--
44+
Required for release automation.
45+
Format: Keep a Changelog.
46+
Categories: Added, Changed, Fixed, Removed.
47+
User-facing notes only. Internal-only PRs (rare) may use the skip-changelog label.
48+
Example:
49+
### Changed
50+
- Switched to action/cache@v3 for build speedup. (Relates to #789)
51+
-->
52+
53+
### Added
54+
55+
- [placeholder]
56+
57+
### Changed
58+
59+
- [placeholder]
60+
61+
### Fixed
62+
63+
- [placeholder]
64+
65+
### Removed
66+
67+
- [placeholder]
68+
69+
<!--
70+
If no user-facing changelog entry is needed, apply the skip-changelog label to this PR.
71+
-->
3072

3173
---
3274

75+
3376
### Checklist (Global DoD / PR)
77+
3478
- [ ] All AC met and demonstrated
3579
- [ ] Tests added/updated (unit/E2E as appropriate)
3680
- [ ] A11y considerations addressed where relevant
3781
- [ ] Docs/readme/changelog updated (if user-facing)
3882
- [ ] Security/perf impact reviewed where relevant
3983
- [ ] Code/design reviews approved
4084
- [ ] CI green; linked issues closed; release notes prepared (if shipping)
41-

‎.github/PULL_REQUEST_TEMPLATE/pr_dep_update.md‎

Lines changed: 53 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1,21 +1,29 @@
11
---
2-
name: "Dependencies/Maintenance PR"
3-
about: "Routine upkeep: dependency bumps, lint/format, low-risk hygiene"
4-
title: "chore(deps): <packages>"
5-
labels: ["status:needs-review", "area:dependencies"]
2+
3+
name: "Dependencies/Maintenance PR"
4+
about: "Routine upkeep: dependency bumps, lint/format, low-risk hygiene"
5+
title: "chore(deps): {packages}"
6+
labels: ["status:needs-review", "area:dependencies"]
7+
68
---
7-
<!-- Note: YAML front matter is parsed by our labeler workflow; GitHub itself does not apply labels from front matter. -->
89

9-
<!-- Suggested additional labels (pick as relevant):
10-
- compat:php
11-
- compat:wordpress
12-
- compat:woocommerce
10+
# Dependencies/Maintenance Pull Request
11+
12+
> This repository enforces changelog, release, and label automation for all PRs and issues.
13+
> See the organisation-wide [Automation Governance & Release Strategy](https://github.com/lightspeedwp/.github/blob/main/AUTOMATION_GOVERNANCE.md) for required rules.
14+
15+
## Linked issues
16+
17+
<!--
18+
List any related issues by number (e.g. closes #123, relates to #789).
1319
-->
1420

21+
Relates to #
22+
1523
## Maintenance summary
1624

17-
- Packages bumped: <list>
18-
- Rationale: <security / compatibility / hygiene>
25+
- Packages bumped: (list)
26+
- Rationale: (security / compatibility / hygiene)
1927

2028
## Impact assessment
2129

@@ -28,14 +36,47 @@ labels: ["status:needs-review", "area:dependencies"]
2836
- [ ] Smoke tests
2937
- [ ] Key pages/components checked
3038

39+
## Changelog
40+
41+
<!--
42+
Required for release automation.
43+
Format: Keep a Changelog.
44+
Categories: Added, Changed, Fixed, Removed.
45+
User-facing notes only. Internal-only PRs (rare) may use the skip-changelog label.
46+
Example:
47+
### Changed
48+
- Updated WooCommerce dependency to v8.1 for compatibility. (Relates to #123)
49+
-->
50+
51+
### Added
52+
53+
- [placeholder]
54+
55+
### Changed
56+
57+
- [placeholder]
58+
59+
### Fixed
60+
61+
- [placeholder]
62+
63+
### Removed
64+
65+
- [placeholder]
66+
67+
<!--
68+
If no user-facing changelog entry is needed, apply the skip-changelog label to this PR.
69+
-->
70+
3171
---
3272

73+
3374
### Checklist (Global DoD / PR)
75+
3476
- [ ] All AC met and demonstrated
3577
- [ ] Tests added/updated (unit/E2E as appropriate)
3678
- [ ] A11y considerations addressed where relevant
3779
- [ ] Docs/readme/changelog updated (if user-facing)
3880
- [ ] Security/perf impact reviewed where relevant
3981
- [ ] Code/design reviews approved
4082
- [ ] CI green; linked issues closed; release notes prepared (if shipping)
41-

0 commit comments

Comments
 (0)