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

pull request template #981

Merged
merged 1 commit into from
Sep 22, 2023
Merged

pull request template #981

merged 1 commit into from
Sep 22, 2023

Conversation

karkir0003
Copy link
Member

@karkir0003 karkir0003 commented Sep 22, 2023

Update PR Template with Reminder to Include Github Issue Number

What user problem are we solving?
Reminding developers to include issue number in their PR description in order to link github issue to PR for DLP project board automation to trigger (ie: when PR merged, linked issue automatically marked as closed).

What solution does this PR provide?
Adding a field in the PR template description for PR author to enter the github issue number using the hashtag

Testing Methodology
See if developer fills out the field in the upcoming PRs

Any other considerations
N/A

Copy link
Contributor

@noah-iversen noah-iversen left a comment

Choose a reason for hiding this comment

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

Approved. Looks good to me.

@karkir0003 karkir0003 added this pull request to the merge queue Sep 22, 2023
Merged via the queue into nextjs with commit f3c7cd5 Sep 22, 2023
9 checks passed
@karkir0003 karkir0003 deleted the pr-template-update branch September 22, 2023 03:45
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