-
Notifications
You must be signed in to change notification settings - Fork 870
/
pull_request_template.md
42 lines (30 loc) · 1.33 KB
/
pull_request_template.md
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
### Description
<!-- Describe what this change achieves-->
### Issues Resolved
<!-- List any issues this PR will resolve. Prefix the issue with the keyword closes, fixes, fix -->
<!-- Example: closes #1234 or fixes <Issue_URL> -->
## Screenshot
<!-- Attach any relevant screenshots. Any change to the UI requires an attached screenshot in the PR Description -->
## Testing the changes
<!--
Please provide detailed steps for validating your changes. This could involve specific commands to run,
pages to visit, scenarios to try or any other information that would help reviewers verify
the functionality of your change
-->
## Changelog
<!--
Add a short but concise sentence about the impact of this pull request. Prefix an entry with the type of change they correspond to: breaking, chore, deprecate, doc, feat, fix, infra, refactor, test.
- fix: Update the graph
- feat: Add a new feature
If this change does not need to added to the changelog, just add a single `skip` line e.g.
- skip
Descriptions following the prefixes must be 100 characters long or less
-->
### Check List
- [ ] All tests pass
- [ ] `yarn test:jest`
- [ ] `yarn test:jest_integration`
- [ ] New functionality includes testing.
- [ ] New functionality has been documented.
- [ ] Update [CHANGELOG.md](./../CHANGELOG.md)
- [ ] Commits are signed per the DCO using --signoff