docs: update the PR template to suggest following the commit message guidelines (#5436)

This commit is contained in:
Stanimira Vlaeva
2018-02-21 11:17:39 +02:00
committed by Alexander Vakrilov
parent 625d801014
commit c7dd02c22e

View File

@@ -1,12 +1,37 @@
To help the rest of the community review your change, please ensure: <!--
We, the rest of the NativeScript community, thank you for your
contribution!
To help the rest of the community review your change, please follow the instructions in the template.
-->
### PR has a meaningful title <!-- PULL REQUEST TEMPLATE -->
A good title is less than 50 characters and starts with a capital <!-- (Update "[ ]" to "[x]" to check a box) -->
letter, similar to a good [Git Commit Message] (http://chris.beams.io/posts/git-commit/).
### The commit message references a specific issue in this repo ## PR Checklist
Fixes/Implements #[Issue Number].
### You have [unit tests](https://github.com/NativeScript/NativeScript/blob/master/running-tests.md) - [ ] The PR title follows our guidelines: https://github.com/NativeScript/NativeScript/blob/master/CONTRIBUTING.md#commit-messages.
if appropriate. - [ ] There is an issue for the bug/feature this PR is for. To avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it.
- [ ] You have signed the [CLA](http://www.nativescript.org/cla).
- [ ] All existing tests are passing: https://github.com/NativeScript/NativeScript/blob/master/DevelopmentWorkflow.md#running-unit-tests.
- [ ] Tests for the changes are included - https://github.com/NativeScript/NativeScript/blob/master/WritingUnitTests.md.
## What is the current behavior?
<!-- Please describe the current behavior that you are modifying, or link to a relevant issue. -->
## What is the new behavior?
<!-- Describe the changes. -->
Fixes/Implements/Closes #[Issue Number].
<!-- If this PR contains a breaking change, please describe the impact and migration path for existing applications below. -->
<!--
BREAKING CHANGES:
[Describe the impact of the changes here.]
Migration steps:
[Provide a migration path for existing applications.]
-->