Skip to content

Update pull request template#5523

Open
emilykl wants to merge 3 commits intomainfrom
update-pr-template
Open

Update pull request template#5523
emilykl wants to merge 3 commits intomainfrom
update-pr-template

Conversation

@emilykl
Copy link
Contributor

@emilykl emilykl commented Feb 23, 2026

Link to issue

Closes #5525

Description of change

  • Streamline PR template and move the long checklists into CONTRIBUTING.md
  • Add "Demo" section to template

The intention is to reduce information overload in the PR template, and clarify the most important items required for a PR description, to make review easier.

Demo

This PR description follows the new proposed template.

Testing strategy

N/A, not a code change

Guidelines

@emilykl emilykl marked this pull request as ready for review February 25, 2026 22:18
@emilykl emilykl changed the title Update pull_request_template.md Update pull request template Feb 25, 2026
@emilykl emilykl force-pushed the update-pr-template branch from 5be243e to b9dfe8d Compare March 2, 2026 17:57
Copy link
Contributor

@camdecoster camdecoster left a comment

Choose a reason for hiding this comment

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

I left a few small comments, but it seems like a good change. If we get better PRs as a result, we should consider rolling the change out to other repos.

Please complete each section below.
-->

## Link to issue
Copy link
Contributor

Choose a reason for hiding this comment

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

My personal preference would be for these to be H3s:

Suggested change
## Link to issue
### Link to issue


<!-- Include any additional context, background, or explanation which doesn't fit in the previous sections. -->

## Guidelines
Copy link
Contributor

Choose a reason for hiding this comment

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

What do you think of including the changelog note here as well? That seems to be one task that often gets missed.

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.

Revise PR template

3 participants