Skip to content

Conversation

@Jibola
Copy link
Contributor

@Jibola Jibola commented Oct 22, 2025

PYTHON-5579

Summary

This pull request adds a new pull request template to the repository to standardize the information provided when submitting PRs. The template guides contributors to include context, a summary, test plans, and checklists for both authors and reviewers, improving clarity and review efficiency.
This PR template is in alignment with our new team PR Template and code reviewing policy.

Changes in this PR

Added a detailed pull request template in .github/workflows/PULL_REQUEST_TEMPLATE.md to ensure consistent PR submissions, including sections for summary, changes, test plan, screenshots, and checklists for both authors and reviewers.

Test Plan

None. Done several times across multiple other repositories.

Screenshots (optional)

image

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?

Checklist for Reviewers: @aclark4life @WaVEV @timgraham

  • 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

Please provide feedback on any changes to this template pertinent to the django-mongodb-backend workflow. All feedback is welcomed and open to discussion.

This template requirement will only apply to PRs made after this PR is approved and merged.

- [ ] Have you checked for spelling & grammar errors?
- [ ] Is all relevant documentation (README or docstring) updated?

## Focus Areas for Reviewer
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can this one be (Optional) ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Absolutely

@timgraham timgraham marked this pull request as draft October 29, 2025 19:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants