Skip to content

Commit 5d2b1b3

Browse files
committed
chore(github): tighten PR and issue templates
- PR template: remove boilerplate, collapse test command to one line - Bug template: shorter, drop redundant fields - Feature template: add acceptance criteria checkbox
1 parent af8625d commit 5d2b1b3

3 files changed

Lines changed: 13 additions & 30 deletions

File tree

.github/ISSUE_TEMPLATE/bug.md

Lines changed: 5 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -4,22 +4,15 @@ about: Something is broken
44
labels: bug
55
---
66

7-
## Describe the bug
8-
9-
<!-- What happened? What did you expect? -->
7+
## What happened
8+
<!-- What did you expect vs what actually occurred? -->
109

1110
## Steps to reproduce
12-
1311
1.
1412
2.
15-
3.
1613

1714
## Environment
15+
- Browser / Device:
16+
- Transport (PeerJS / Gun.js / Auto):
1817

19-
- Browser:
20-
- Device:
21-
- Transport mode (PeerJS / Gun.js / Auto):
22-
23-
## Additional context
24-
25-
<!-- Screenshots, console errors, etc. -->
18+
<!-- Screenshots or console errors welcome -->

.github/ISSUE_TEMPLATE/feature.md

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -5,13 +5,10 @@ labels: enhancement
55
---
66

77
## Problem
8-
9-
<!-- What problem are you trying to solve? -->
8+
<!-- What are you trying to do that you can't do today? -->
109

1110
## Proposed solution
12-
1311
<!-- What would you like to see? -->
1412

15-
## Alternatives considered
16-
17-
<!-- Any other approaches you thought about? -->
13+
## Acceptance criteria
14+
- [ ]

.github/PULL_REQUEST_TEMPLATE.md

Lines changed: 5 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1,24 +1,17 @@
1-
## What
2-
31
Closes #<!-- issue number -->
42

3+
## What
54
<!-- One-line summary of the change. -->
65

76
## Why
7+
<!-- What problem does this solve? -->
88

9-
<!-- Context: what problem does this solve, or what feature does it add? -->
10-
11-
## How
12-
13-
<!-- Brief description of the approach. Call out any tricky decisions. -->
9+
## Notes
10+
<!-- Tricky decisions, follow-up work, manual testing steps. Delete if none. -->
1411

1512
## Checklist
16-
1713
- [ ] PR title follows conventional commits — `type(scope): description`
1814
Scopes: `admin` · `player` · `transport` · `db` · `ui` · `pwa` · `deps` · `release`
1915
- [ ] Tests added or updated
20-
- [ ] `npm run test` passes locally
21-
- [ ] `npm run build` passes locally
22-
- [ ] `npm run lint` passes locally
16+
- [ ] `npm test && npm run build && npm run lint` pass locally
2317
- [ ] Docs updated if behaviour changed
24-
- [ ] No `console.log` left in production paths

0 commit comments

Comments
 (0)