FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Update PULL_REQUEST_TEMPLATE.md to include issue cross-reference. by chahak13 · Pull Request #25737 · matplotlib/matplotlib · GitHub

Repository navigation

Update PULL_REQUEST_TEMPLATE.md to include issue cross-reference. - #25737

Merged
QuLogic merged 2 commits into
matplotlib:mainfrom
chahak13:patch-1
Apr 22, 2023
Merged

QuLogic merged 2 commits into
matplotlib:mainfrom
chahak13:patch-1

Conversation

Copy link
Copy Markdown
Contributor

This changes the PR template to prompt/remind the user to link the issue it resolves. This is to make sure that it reduces duplicate PRs, as having a back link would make it easier to know that there's work being done on a particular issue.

PR Summary

PR Checklist

Documentation and Tests

  • [N/A] Has pytest style unit tests (and pytest passes)
  • [N/A] Documentation is sphinx and numpydoc compliant (the docs should build without error).
  • [N/A] New plotting related features are documented with examples.

Release Notes

  • [N/A] New features are marked with a .. versionadded:: directive in the docstring and documented in doc/users/next_whats_new/
  • [N/A] API changes are marked with a .. versionchanged:: directive in the docstring and documented in doc/api/next_api_changes/
  • [N/A] Release notes conform with instructions in next_whats_new/README.rst or next_api_changes/README.rst

This changes the PR template to prompt/remind the user to link the issue it resolves. This is to make sure that it reduces duplicate PRs, as having a back link would make it easier to know that there's work being done on a particular issue.

story645 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

I'm wondering also if we can use githubs form schema and make this a required text area.

Comment thread .github/PULL_REQUEST_TEMPLATE.md Outdated

Copy link
Copy Markdown
Contributor Author

I'm wondering also if we can use githubs form schema and make this a required text area.

It doesn't have to be necessary because that might hamper just normal MNT PRs or something that's not related to the issue.

QuLogic added this to the v3.8.0 milestone Apr 22, 2023
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants


Back | FazBrowse Home | New Git URL