-
Notifications
You must be signed in to change notification settings - Fork 1.1k
PYTHON-5401: Revise pull request template for better structure #2626
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from all commits
Commits
Show all changes
5 commits
Select commit
Hold shift + click to select a range
97b0f0c
PYTHON-5401: Revise pull request template for better structure
Jibola efe4893
Merge branch 'master' into PYTHON-5401
Jibola fb4b2c2
Removed the link on JIRA TICKET
Jibola 10ba0cf
Add "note on ai contributions"
Jibola 1a640b1
Merge branch 'master' into PYTHON-5401
Jibola File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,38 +1,33 @@ | ||
| <!-- Thanks for contributing! --> | ||
| <!-- Please ensure that the title of the PR is in the following form: | ||
| [Issue Type]-[Issue Key]: Issue Title | ||
| [JIRA TICKET]: Issue Title | ||
|
|
||
| If you are an external contributor and there is no JIRA ticket associated with your change, then use your best judgement | ||
| for the PR title. A MongoDB employee will create a JIRA ticket and edit the name and links as appropriate. | ||
|
|
||
| Note on AI Contributions: | ||
| We do not accept pull requests that are primarily or substantially generated by AI tools (ChatGPT, Copilot, etc.). | ||
| All contributions must be written and understood by human contributors. | ||
| --> | ||
| [Issue Key](https://jira.mongodb.org/browse/%7BISSUE_KEY%7D) | ||
| ## Summary | ||
| <!-- What conceptually 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. --> | ||
| [JIRA TICKET] | ||
|
|
||
| ## Changes in this PR | ||
| <!-- 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? --> | ||
|
|
||
| ## Testing Plan | ||
| ## Test Plan | ||
| <!-- 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. --> | ||
|
|
||
| ### Screenshots (optional) | ||
| <!-- Usually a great supplement to a test plan, especially if this requires local testing. --> | ||
|
|
||
| ## Checklist | ||
| <!-- Do not delete the items provided on this checklist. --> | ||
|
|
||
| ### Checklist for Author | ||
| - [ ] Did you update the changelog (if necessary)? | ||
| - [ ] Is the intention of the code captured in relevant tests? | ||
| - [ ] If there are new TODOs, has a related JIRA ticket been created? | ||
| - [ ] Is there test coverage? | ||
| - [ ] Is any followup work tracked in a JIRA ticket? If so, add link(s). | ||
|
|
||
| ### Checklist for Reviewer {@primary_reviewer} | ||
| ### Checklist for Reviewer | ||
| - [ ] Does the title of the PR reference a JIRA Ticket? | ||
| - [ ] Do you fully understand the implementation? (Would you be comfortable explaining how this code works to someone else?) | ||
| - [ ] Have you checked for spelling & grammar errors? | ||
| - [ ] Is all relevant documentation (README or docstring) updated? | ||
|
|
||
| ## Focus Areas for Reviewer (optional) | ||
| <!-- List any complex portion of code you believe needs additional scrutiny and explain why. --> | ||
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.