-
Notifications
You must be signed in to change notification settings - Fork 7.2k
ignore: add PR template #7391
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
ignore: add PR template #7391
Conversation
|
Hey! Your PR title Please update it to start with one of:
Where See CONTRIBUTING.md for details. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR adds a minimal pull request template to help standardize PR descriptions across the repository. The template prompts contributors to describe what their PR does and how they verified their changes work.
Key Changes:
- Adds a new PR template file in the
.githubdirectory with two prompting sections
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| @@ -0,0 +1,3 @@ | |||
| ### What does this PR do? | |||
|
|
|||
| ### How did you verify your code works? | |||
Copilot
AI
Jan 8, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Consider adding a section for UI changes to prompt contributors to include screenshots or videos. According to CONTRIBUTING.md (lines 165-167), UI changes should include visual evidence showing before and after states. This could be a conditional section like "### Screenshots/Videos (if applicable for UI changes)".
| ### How did you verify your code works? | |
| ### How did you verify your code works? | |
| ### Screenshots/Videos (if applicable for UI changes) |
| @@ -0,0 +1,3 @@ | |||
| ### What does this PR do? | |||
|
|
|||
Copilot
AI
Jan 8, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The PR template is missing a section to prompt contributors to link to an existing issue. According to CONTRIBUTING.md (lines 152-157), there's an "Issue First Policy" stating that all PRs must reference an existing issue using "Fixes #123" or "Closes #123". Consider adding a section like "### What issue does this PR address?" to ensure contributors follow this requirement.
| ### What issue does this PR address? | |
| <!-- Link to the issue using "Fixes #123" or "Closes #123". --> |
|
The following comment was made by an LLM, it may be inaccurate: Based on my search, I found one potentially related PR: PR #7013: fix(tool): require PR template check before creating PRs This PR is related because it deals with PR template validation/requirements. While PR #7391 adds a new PR template and #7013 enforces checking for PR templates before creation, they may be complementary changes or there could be some overlap in scope depending on the implementation details. The other results appear to be unrelated to the PR template feature. |
Summary