Skip to content

Add Pull Request (PR) Template#37

Merged
SrishtiSonam merged 9 commits intoSrishtiSonam:mainfrom
dimpal-yadav:main
Oct 12, 2025
Merged

Add Pull Request (PR) Template#37
SrishtiSonam merged 9 commits intoSrishtiSonam:mainfrom
dimpal-yadav:main

Conversation

@dimpal-yadav
Copy link
Contributor

Which issue does this PR close?
Closes #35

Rationale for this change
I have updated the .github/pull_request_template.md to make it more professional, comprehensive, and reviewer-friendly.
The updated template now includes:

Clear headings and visual icons for readability
Step-by-step testing instructions section
Reviewer guidance / notes section
Breaking changes section
Impact assessment and priority labels
Enhanced checklist to ensure nothing is missed
This will help contributors create better PRs and allow maintainers to review changes efficiently.

What changes are included in this PR?
Updated .github/pull_request_template.md with the new format

Added sections for:
PR Type
Related Issue
Rationale / Motivation
Changes included
Testing instructions
Impact assessment
Reviewer notes
Breaking changes
Priority / Impact level
Improved formatting with emojis and nested bullet points for clarity

Are these changes tested?
N/A – This is a documentation / template update. No code changes are included.
Verified that the template renders correctly when creating a new PR.

Are there any user-facing changes?
No, this is an internal improvement for contributors and reviewers.

Screenshots
N/A – no visual changes.

Checklist
My code follows the project’s coding style and guidelines
My changes do not introduce new bugs or break existing functionality
I have linked the related issue number in this PR (if applicable)
I have added meaningful comments and documentation where necessary
I have updated relevant tests or added new ones to cover my changes (N/A for template update)
I have verified that all existing and new tests pass successfully

@github-actions
Copy link

github-actions bot commented Oct 6, 2025

Thanks for creating a PR for your Issue! ☺️

We'll review it as soon as possible.
In the meantime, please double-check the file changes and ensure that all commits are accurate.

If there are any unresolved review comments, feel free to resolve them. 🙌🏼

@SrishtiSonam SrishtiSonam merged commit f6d300a into SrishtiSonam:main Oct 12, 2025
@dimpal-yadav
Copy link
Contributor Author

Can you add gssoc and level label on it

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.

Add Pull Request (PR) Template

2 participants