diff options
| author | Joram Wilander <jwawilander@gmail.com> | 2016-07-06 12:25:41 -0400 |
|---|---|---|
| committer | Corey Hulen <corey@hulen.com> | 2016-07-06 08:25:41 -0800 |
| commit | ccbf06dca957f685a264c71643716933c3b13c6d (patch) | |
| tree | 43836dc70ebd306df7c63792b5a9fb101464df2e /PULL_REQUEST_TEMPLATE.md | |
| parent | 2b60c96994a99676ddaaff7eabaef061c0075755 (diff) | |
| download | chat-ccbf06dca957f685a264c71643716933c3b13c6d.tar.gz chat-ccbf06dca957f685a264c71643716933c3b13c6d.tar.bz2 chat-ccbf06dca957f685a264c71643716933c3b13c6d.zip | |
Updated pull request template (#3504)
Diffstat (limited to 'PULL_REQUEST_TEMPLATE.md')
| -rw-r--r-- | PULL_REQUEST_TEMPLATE.md | 22 |
1 files changed, 14 insertions, 8 deletions
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.) |