-
Notifications
You must be signed in to change notification settings - Fork 0
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
Create pull_request_template.md #36
Changes from all commits
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,27 @@ | ||||||||||||||||||||||
<!-- Please refer to our contributing docs for any questions on submitting a pull request --> | ||||||||||||||||||||||
<!-- Issues are required for both bug fixes and features. --> | ||||||||||||||||||||||
Resolves #ISSUE_NUMBER | ||||||||||||||||||||||
|
||||||||||||||||||||||
---- | ||||||||||||||||||||||
|
||||||||||||||||||||||
### Before the change? | ||||||||||||||||||||||
<!-- Please describe the current behaviour that you are modifying. --> | ||||||||||||||||||||||
|
||||||||||||||||||||||
* | ||||||||||||||||||||||
Comment on lines
+7
to
+10
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. Improve clarity and fix list formatting. Clarify the placeholder to guide contributors better and fix the list formatting. ### Before the change
<!-- Please describe the current behavior that you are modifying. -->
*
ToolsGitHub Check: Codacy Static Code Analysis
|
||||||||||||||||||||||
|
||||||||||||||||||||||
### After the change? | ||||||||||||||||||||||
<!-- Please describe the behaviour or changes that are being added by this PR. --> | ||||||||||||||||||||||
|
||||||||||||||||||||||
* | ||||||||||||||||||||||
Comment on lines
+12
to
+15
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. Improve clarity and fix list formatting. Clarify the placeholder to guide contributors better and fix the list formatting. ### After the change
<!-- Please describe the behavior or changes that are being added by this PR. -->
* Committable suggestion
Suggested change
|
||||||||||||||||||||||
|
||||||||||||||||||||||
### Pull request checklist | ||||||||||||||||||||||
- [ ] Tests for the changes have been added (for bug fixes/features) | ||||||||||||||||||||||
Check notice on line 18 in .github/pull_request_template.md Codacy Production / Codacy Static Code Analysis.github/pull_request_template.md#L18
|
||||||||||||||||||||||
- [ ] Docs have been reviewed and added/updated if needed (for bug fixes/features) | ||||||||||||||||||||||
Comment on lines
+17
to
+19
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. Fix list formatting. Ensure the list items are correctly formatted and surrounded by blank lines. ### Pull request checklist
* [ ] Tests for the changes have been added (for bug fixes/features)
* [ ] Docs have been reviewed and added/updated if needed (for bug fixes/features) Committable suggestion
Suggested change
ToolsGitHub Check: Codacy Static Code Analysis
|
||||||||||||||||||||||
|
||||||||||||||||||||||
### Does this introduce a breaking change? | ||||||||||||||||||||||
<!-- If this introduces a breaking change, make sure to note it here and what the impact might be --> | ||||||||||||||||||||||
|
||||||||||||||||||||||
- [ ] Yes | ||||||||||||||||||||||
- [ ] No | ||||||||||||||||||||||
Comment on lines
+21
to
+25
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. Improve clarity, make the section more explicit, and fix list formatting. Clarify the placeholder, make the section more explicit, and ensure the list items are correctly formatted. ### Does this introduce a breaking change?
<!-- If this introduces a breaking change, make sure to note it here and what the impact might be -->
* [ ] Yes
* [ ] No Committable suggestion
Suggested change
ToolsGitHub Check: Codacy Static Code Analysis
|
||||||||||||||||||||||
|
||||||||||||||||||||||
---- | ||||||||||||||||||||||
Comment on lines
+1
to
+27
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. CODE REVIEW
<!-- Refer to contributing docs for questions about submitting a PR -->
<!-- Issues are required for both bug fixes and features. -->
Resolves #ISSUE_NUMBER
----
### Before the change
<!-- Describe the current behavior being modified. -->
*
### After the change
<!-- Describe the new behavior implemented by this PR. -->
*
### Pull request checklist
- [ ] Tests added for changes (bug fixes/features)
- [ ] Documentation reviewed and updated if needed
### Breaking change
<!-- Does this introduce a breaking change? Note the impact. -->
- [ ] Yes
- [ ] No
---- |
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.
Add a top-level heading and improve clarity.
The first line should be a top-level heading for better structure. Additionally, clarify the comments to guide contributors better.
+ # Pull Request Template <!-- Please refer to our contributing docs for any questions on submitting a pull request --> <!-- Issues are required for both bug fixes and features. --> Resolves #ISSUE_NUMBER
Committable suggestion
Tools
GitHub Check: Codacy Static Code Analysis