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.

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 templates #2217

Merged
merged 1 commit into from
Aug 26, 2024
Merged

Add pull request templates #2217

merged 1 commit into from
Aug 26, 2024

Conversation

EwoutH
Copy link
Member

@EwoutH EwoutH commented Aug 17, 2024

Added GitHub pull request templates for bug fixes and feature additions to help new contributors format their PRs and help maintainers to review these.

The PR templates where modelled on the large PRs that were merged into Mesa recently.

In general, I don't like pushing things too hard into a mold, since each PR is different and might benefit from a different structure, but I feel these two are a good starting point, especially for new contributors.

See the docs Creating a pull request template for your repository for more information.

Added GitHub pull request templates for bug fixes and feature additions to help new contributors format their PRs and help maintainers to review these.

The PR templates where modelled on the large PRs that were merged into Mesa recently.

See https://docs.github.com/en/communities/using-templates-to-encourage-useful-issues-and-pull-requests/creating-a-pull-request-template-for-your-repository
@EwoutH EwoutH added the maintenance Release notes label label Aug 17, 2024
@quaquel
Copy link
Member

quaquel commented Aug 21, 2024

I like these as templates, but indeed it should not become a straightjacket.

Copy link
Member

@tpike3 tpike3 left a comment

Choose a reason for hiding this comment

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

Thanks @EwoutH LGTM

@tpike3 tpike3 merged commit 50b2964 into main Aug 26, 2024
2 checks passed
@EwoutH EwoutH deleted the pr_templates branch September 20, 2024 09:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance Release notes label
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants