|
1 | | -[Issue Key](https://jira.mongodb.org/browse/{ISSUE_KEY}) |
| 1 | +<!-- Thanks for contributing! --> |
| 2 | +<!-- Please ensure that the title of the PR is in the following form: |
| 3 | +[JIRA TICKET]: Issue Title |
2 | 4 |
|
3 | | -## Summary |
4 | | - |
5 | | -<!-- What is this PR introducing? If context is already provided from the JIRA ticket, still place it in the Pull Request as you should not make the reviewer do digging for a basic summary. --> |
| 5 | +If you are an external contributor and there is no JIRA ticket associated with your change, then use your best judgement |
| 6 | +for the PR title. A MongoDB employee will create a JIRA ticket and edit the name and links as appropriate. |
| 7 | +--> |
| 8 | +[JIRA TICKET](https://jira.mongodb.org/browse/%7BJIRA_TICKET%7D) |
6 | 9 |
|
7 | 10 | ## Changes in this PR |
8 | | - |
9 | | -<!-- What changes did you make to the code? What new APIs (public or private) were added, removed, or edited to generate the desired outcome explained in the above summary? --> |
| 11 | +<!-- What changes did you make to the code? What new APIs (public or private) were added, removed, or edited to generate |
| 12 | +the desired outcome explained in the above summary? --> |
10 | 13 |
|
11 | 14 | ## Test Plan |
12 | | - |
13 | | -<!-- How did you test the code? If you added unit tests, you can say that. If you didn’t introduce unit tests, explain why. All code should be tested in some way – so please list what your validation strategy was. --> |
14 | | - |
15 | | -### Screenshots (optional) |
16 | | - |
17 | | -<!-- Usually a great supplement to a test plan, especially if this requires local testing. --> |
| 15 | +<!-- How did you test the code? If you added unit tests, you can say that. If you didn’t introduce unit tests, explain why. |
| 16 | +All code should be tested in some way – so please list what your validation strategy was. --> |
18 | 17 |
|
19 | 18 | ## Checklist |
20 | | - |
21 | | -<!-- Do not delete the items provided on this checklist --> |
| 19 | +<!-- Do not delete the items provided on this checklist. --> |
22 | 20 |
|
23 | 21 | ### Checklist for Author |
24 | | - |
25 | 22 | - [ ] Did you update the changelog (if necessary)? |
26 | | -- [ ] Is the intention of the code captured in relevant tests? |
27 | | -- [ ] If there are new TODOs, has a related JIRA ticket been created? |
28 | | - |
29 | | -### Checklist for Reviewer {@primary_reviewer} |
| 23 | +- [ ] Is there test coverage? |
| 24 | +- [ ] Is any followup work tracked in a JIRA ticket? If so, add link(s). |
30 | 25 |
|
| 26 | +### Checklist for Reviewer |
31 | 27 | - [ ] Does the title of the PR reference a JIRA Ticket? |
32 | 28 | - [ ] Do you fully understand the implementation? (Would you be comfortable explaining how this code works to someone else?) |
33 | | -- [ ] Have you checked for spelling & grammar errors? |
34 | 29 | - [ ] Is all relevant documentation (README or docstring) updated? |
35 | | - |
36 | | -## Focus Areas for Reviewer (optional) |
37 | | - |
38 | | -<!-- List any complex portion of code you believe needs particular scrutiny and explain why. --> |
0 commit comments