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

new PR template #2191

Merged
merged 3 commits into from
Sep 5, 2024
Merged

new PR template #2191

merged 3 commits into from
Sep 5, 2024

Conversation

mhratson
Copy link
Contributor

@mhratson mhratson commented Sep 1, 2024

Summary

Why: Improve PR quality and review-ability.
What: modifies current PR template to be structured and require more details when submitting PRs.

Expected Behavior

PR must come with sufficient details to address or explain the issue.

Actual Behavior

PR template only requires link to the issue:

This PR resolves #<Replace-Me-With-The-Issue-Number-Addressed-By-This-PR>.

Steps to reproduce

  1. either create a new PR or
  2. see the current template

Known Workarounds

People voluntarily provide additional details

Additional Evidence

  • n/a

Categorization

  • refactor

@mhratson mhratson changed the title new PR/Issue templates new PR template Sep 5, 2024
@mhratson mhratson merged commit a298df8 into main Sep 5, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants