Skip to content

Add PR template and update contributor docs#691

Merged
edwardchalstrey1 merged 8 commits intomasterfrom
PR-practices
Dec 16, 2025
Merged

Add PR template and update contributor docs#691
edwardchalstrey1 merged 8 commits intomasterfrom
PR-practices

Conversation

@edwardchalstrey1
Copy link
Copy Markdown
Member

@edwardchalstrey1 edwardchalstrey1 commented Dec 12, 2025

Issues closed by this PR

How to review this PR

See the comment below which includes how the PR template will look by default and review the contributor page at https://gambitproject--691.org.readthedocs.build/en/691/developer.contributing.html#contributing-code

@edwardchalstrey1 edwardchalstrey1 changed the title Add PR template Add PR template and update contributor docs Dec 12, 2025
@edwardchalstrey1
Copy link
Copy Markdown
Member Author

Thanks for contributing to Gambit! Before you submitting or reviewing a pull request, check out our guidelines for contributing.

The person submitting the PR should ensure it has an informative title and update the headers below, before marking the PR as ready for review and assigning reviewers.

Issues closed by this PR

Add any issues that are being closed by this PR to the list. Use "Closes" or another keyword followed by the issue number.

  • Closes #ISSUE_NUMBER

Description of the changes in this PR

Update the description of the changes made in this PR. You can delete this section if the changes are already fully described in the linked issues.

This PR ...

How to review this PR

The gambit repository contains source code and documentation for several different components. Explain how a reviewer should approach reviewing the specific changes made in this PR.

For example, you might include instructions like:

  • "Review code changes and ensure tests cover edge cases"
  • "Rebuild the GUI from this branch, then test features X, Y, Z work as expected"
  • "Click the link to the documentation page and sense check by reading"

@edwardchalstrey1 edwardchalstrey1 marked this pull request as ready for review December 12, 2025 11:04
@edwardchalstrey1 edwardchalstrey1 merged commit 5a10e7f into master Dec 16, 2025
28 checks passed
@edwardchalstrey1 edwardchalstrey1 deleted the PR-practices branch December 16, 2025 14:22
@github-project-automation github-project-automation bot moved this from In review to Done in Ed Chalstrey Gambit priorities Dec 16, 2025
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.

[Documentation]: Pull request and branching team practices

1 participant