-
-
Notifications
You must be signed in to change notification settings - Fork 15
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
Add pull request template #16
base: main
Are you sure you want to change the base?
Changes from 1 commit
3c9886c
9fc5d23
31b0804
32cecb7
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change | ||||
---|---|---|---|---|---|---|
@@ -0,0 +1,35 @@ | ||||||
# Pull Request Template β¨ | ||||||
|
||||||
## π Description | ||||||
<!-- Provide a concise yet clear summary of what this PR changes or adds. --> | ||||||
- What problem does it solve? | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Minor suggestion because not all PR's resolve a problem (some add features).
Suggested change
|
||||||
- Why is it necessary? | ||||||
- Highlight any interesting or unique aspects of the change. | ||||||
|
||||||
--- | ||||||
|
||||||
## π Related Issue | ||||||
<!-- Link the issue this PR addresses and include a brief explanation if needed. --> | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Consider adding a documentation here about the keyword usage within pull requests and how this impacts the language used below within this section. This link might be informative within the HTML comment. https://docs.github.com/en/get-started/writing-on-github/working-with-advanced-formatting/using-keywords-in-issues-and-pull-requests |
||||||
Closes/Fixes: # | ||||||
|
||||||
--- | ||||||
|
||||||
## β Checklist | ||||||
Before submitting your PR, ensure the following: | ||||||
- [ ] π **Clear Description**: I have provided a detailed description of the changes made. | ||||||
- [ ] ποΈ **Issue Linked**: I have linked the correct issue and included its number above. | ||||||
- [ ] π **Functionality Tested**: I have thoroughly tested the changes, and they work as intended. | ||||||
- [ ] π **Duplicate Check**: I have ensured there are no other PRs addressing the same issue or feature. | ||||||
- [ ] π **Documentation Updated**: I have updated relevant documentation (if applicable). | ||||||
- [ ] β¨ **Creative Touch**: Iβve added value beyond the minimal requirements to enhance quality and readability. | ||||||
|
||||||
--- | ||||||
|
||||||
## π‘ Notes to Reviewers (Optional) | ||||||
<!-- Add any additional context, insights, or areas for reviewers to focus on. --> | ||||||
- Highlight complex parts or reasoning behind specific decisions. | ||||||
- Suggest areas where feedback is especially welcome. | ||||||
|
||||||
--- | ||||||
|
||||||
### π Thank you for contributing! Together, we make this project better. | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I really like the sentiment of this section! However, it could get repetitive if seen in multiple PR's. Consider moving the statement to an HTML comment near the beginning of the template as a message to the PR author. |
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 removing this title, which would otherwise show up in the template each time it is used.