From 606d65735c2f737bb1b655358e2218d07aa32ea7 Mon Sep 17 00:00:00 2001 From: choco-sync Date: Tue, 2 Jan 2024 09:57:41 +0000 Subject: [PATCH] (maint) synced local '.github/PULL_REQUEST_TEMPLATE.md' with remote '.github/PULL_REQUEST_TEMPLATE.md' --- .github/PULL_REQUEST_TEMPLATE.md | 84 +++++++++++++++++++++++--------- 1 file changed, 60 insertions(+), 24 deletions(-) diff --git a/.github/PULL_REQUEST_TEMPLATE.md b/.github/PULL_REQUEST_TEMPLATE.md index aa3787e2de3..e75161c0916 100644 --- a/.github/PULL_REQUEST_TEMPLATE.md +++ b/.github/PULL_REQUEST_TEMPLATE.md @@ -1,31 +1,67 @@ - -## Bug + - -Fixes: +Ensure you have read over [CONTRIBUTING.md](./CONTRIBUTING.md). We provide VERY defined guidance (as such, we strongly adhere to it). -Regression? Last working version: +A summary of our expectations: + - You are not submitting a pull request from your MASTER / MAIN branch. + - You are able to sign the Contributor License Agreement (CLA). + - YOUR GIT COMMIT MESSAGE FORMAT IS EXTREMELY IMPORTANT. We have a very defined expectation for this format and are sticklers about it. Really, READ the entire Contributing document. It will save you and us pain. + - Do not reformat code, it makes it hard to see what has changed. Leave the formatting to us. -## Description - +THANKS! We appreciate you reading the entire Contributing document and not just scanning through it. -## PR Checklist +Name your pull request appropriately: give it a sentence that reads well enough for anyone seeing this. -- [ ] PR has a meaningful title -- [ ] PR has a linked issue. -- [ ] Described changes +if you were explaining it to somebody else. This helps others to understand the reasons for the pull request and for it to be searchable in future. -- **Tests** - - [ ] Automated tests added - - **OR** - - - [ ] Test exception - - **OR** - - [ ] N/A +Please do not remove any of the headings. +If a heading is not applicable then enter N/A: Why it's not applicable -- **Documentation** - - - [ ] Documentation PR or issue filled - - **OR** - - [ ] N/A +Make sure you have raised an issue for this pull request before continuing. + +Please remove all comments before submitting. +--> + +## Description Of Changes + + +## Motivation and Context + + +## Testing + +### Operating Systems Testing + + +## Change Types Made + + +* [ ] Bug fix (non-breaking change). +* [ ] Feature / Enhancement (non-breaking change). +* [ ] Breaking change (fix or feature that could cause existing functionality to change). +* [ ] Documentation changes. +* [ ] PowerShell code changes. + +## Change Checklist + +* [ ] Requires a change to the documentation. +* [ ] Documentation has been updated. +* [ ] Tests to cover my changes, have been added. +* [ ] All new and existing tests passed? +* [ ] PowerShell code changes: PowerShell v2 compatibility checked? + +## Related Issue + + +Fixes # + +