Skip to content
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.

Sign up for GitHub

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

Fix typos in pull request template #4

Merged
merged 2 commits into from
Dec 21, 2024
+8 −9
Merged

Fix typos in pull request template #4

Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
db9fba9
fix: typos pull request template
leahaeusel Dec 19, 2024
d4b0f89
fix: remove updating README from PR template
leahaeusel Dec 20, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 8 additions & 8 deletions .github/pull_request_template.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,11 +1,11 @@
## Description and Context
<!--
Provide a brief and concise description of your proposed change. Why is this change required? What problem does it solve?
Provide a brief and concise description of your proposed change. Why is this change required? What problem does it solve?
-->

## Related Issues and Merge Requests
## Related Issues and Pull Requests
<!--
If applicable, let us know how this merge request is related to any other open issues or pull requests:
If applicable, let us know how this pull request is related to any other open issues or pull requests:
-->
* Closes
* Blocks
Expand All @@ -24,23 +24,23 @@ Feel free to provide further information if useful or necessary.

## Checklist
<!--
Go over all the following points, and put an `x` in all the boxes that apply. If you are unsure about any of these, please ask; we are here to help.
Go over all the following points, and put an `X` in all the boxes that apply. If you are unsure about any of these, please ask; we are here to help.
-->
- [ ] My commit messages mention the appropriate issue numbers (advised but optional).
- [ ] I mention the appropriate issue numbers in this MR.
- [ ] I mention the appropriate issue numbers in this PR.
- [ ] I updated documentation where necessary.
- [ ] I updated the README where necessary
- [ ] I updated the README.md where necessary
- [ ] I have added as (few,) small and fast tests as possible to cover my changes.
leahaeusel marked this conversation as resolved.
Show resolved Hide resolved

## Additional Information
<!--
Is there anything else your fellow developers need to know in evaluating this merge request?
Is there anything else your fellow developers need to know in evaluating this pull request?
Feel free to add supplementary material here (e.g. screen output, log files, screenshots)
-->

## Interested Parties
<!--
If there's anyone you think should be looped in on this merge request, feel free to @mention them here. In particular, @mention possible reviewers as well as the maintainers of all the files you've touched.
If there's anyone you think should be looped in on this pull request, feel free to @mention them here. In particular, @mention possible reviewers as well as the maintainers of all the files you've touched.
-->

Possible reviewers:
Expand Down
Loading