From ccbf06dca957f685a264c71643716933c3b13c6d Mon Sep 17 00:00:00 2001 From: Joram Wilander Date: Wed, 6 Jul 2016 12:25:41 -0400 Subject: Updated pull request template (#3504) --- PULL_REQUEST_TEMPLATE.md | 22 ++++++++++++++-------- 1 file changed, 14 insertions(+), 8 deletions(-) (limited to 'PULL_REQUEST_TEMPLATE.md') diff --git a/PULL_REQUEST_TEMPLATE.md b/PULL_REQUEST_TEMPLATE.md index cea063635..4270a1d17 100644 --- a/PULL_REQUEST_TEMPLATE.md +++ b/PULL_REQUEST_TEMPLATE.md @@ -1,13 +1,19 @@ Please make sure you've read the [pull request](http://docs.mattermost.com/developer/contribution-guide.html#preparing-a-pull-request) section of our [code contribution guidelines](http://docs.mattermost.com/developer/contribution-guide.html). -#### Summary -Brief description of what this pull request does +When filling in a section please remove the help text and the above text. -#### Unit Tests -Have you updated the unit tests or added new ones to cover your changes? If not, you probably should +#### Summary +[A brief description of what this pull request does.] -#### Issue/Ticket Link -Please link the GitHub issue or Jira ticket this PR addresses +#### Ticket Link +[Please link the GitHub issue or Jira ticket this PR addresses.] -#### Has Critical Changes -Does your PR touch critical sections of the codebase including, but not limited to, authentication, authorization or permissions? +#### Checklist +[Place an '[x]' (no spaces) in all applicable fields.] +- [ ] Added or updated unit tests (required for all new features) +- [ ] Added API documentation (required for all new APIs) +- [ ] Has driver changes that have been merged and package.json updated +- [ ] Has enterprise changes (please link) +- [ ] Has UI changes +- [ ] Includes text changes and localization files updated +- [ ] Touches critical sections of the codebase (auth, upgrade, etc.) -- cgit v1.2.3-1-g7c22