Skip to content

Commit 6b9026a

Browse files
⅄ trunk → 25-11-05-difftool
2 parents 013b1df + 3fe9c3e commit 6b9026a

File tree

1 file changed

+15
-11
lines changed

1 file changed

+15
-11
lines changed

.github/pull_request_template.md

Lines changed: 15 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -1,16 +1,14 @@
1-
**Choose your PR title well:** Your pull request title is what's used to create release notes, so please make it descriptive of the change itself, which may be different from the initial motivation to make the change.
2-
31
## Overview
42

5-
What does this change accomplish and why?
6-
i.e. How does it change the user experience?
7-
i.e. What was the old behavior/API and what is the new behavior/API?
3+
- [ ] What does this change accomplish and why?
4+
- [ ] i.e. How does it change the user experience?
5+
- [ ] i.e. What was the old behavior/API and what is the new behavior/API?
86

9-
Feel free to include "before and after" examples if appropriate. (You can copy/paste screenshots directly into this editor.)
7+
- [ ] Include "before and after" examples if appropriate. (You can copy/paste screenshots directly into this editor.)
108

11-
If relevant, which Github issues does it close? (See [closing-issues-using-keywords](https://help.github.com/en/enterprise/2.16/user/github/managing-your-work-on-github/closing-issues-using-keywords).)
9+
- [ ] List any Github issues that this PR closes, in [closing-issues-using-keywords](https://help.github.com/en/enterprise/2.16/user/github/managing-your-work-on-github/closing-issues-using-keywords) format.
1210

13-
## Implementation notes
11+
## Implementation approach and notes
1412

1513
How does it accomplish it, in broad strokes? i.e. How does it change the Haskell codebase?
1614

@@ -21,12 +19,18 @@ What could have been done differently, but wasn't? And why?
2119

2220
## Test coverage
2321

24-
Have you included tests (which could be a transcript) for this change, or is it somehow covered by existing tests?
22+
- [ ] Have you included tests (which could be a transcript) for this change, or is it somehow covered by existing tests?
2523

26-
Would you recommend improving the test coverage (either as part of this PR or as a separate issue) or do you think it’s adequate?
24+
- [ ] Would you recommend improving the test coverage (either as part of this PR or as a separate issue) or do you think it’s adequate?
2725

28-
If you only tested by hand, because that's all that's practical to do for this change, mention that.
26+
- [ ] If you only tested by hand, because that's all that's practical to do for this change, mention that. Include screenshots.
2927

3028
## Loose ends
3129

3230
Link to related issues that address things you didn't get to. Stuff you encountered on the way and decided not to include in this PR.
31+
32+
## Final checklist
33+
34+
- [ ] **Choose your PR title well:** Your pull request title is what's used to create release notes, so please make it descriptive of the change itself, which may be different from the initial motivation to make the change.
35+
- [ ] **Update your PR description** if the specifics of the PR have changed over time.
36+
- [ ] **Include transcripts or screenshots** that demonstrate the changed behavior.

0 commit comments

Comments
 (0)