From f7323493da1a5f278916889e1c0ab44c83dfc248 Mon Sep 17 00:00:00 2001 From: it33 Date: Wed, 16 Mar 2016 00:04:37 -0700 Subject: [PATCH] Adding step to complete CLA + minor formatting --- CONTRIBUTING.md | 15 +++++++-------- 1 file changed, 7 insertions(+), 8 deletions(-) diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index feb1ed68..e8150577 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -27,13 +27,12 @@ Following points are very helpful to understand the problem. Please see http://www.mattermost.org/feature-requests/ . ## Pull request -PR is welcome. Thank you for your great works! +Pull requests are welcome. Thank you for your great work! -When you edit the code, please run `npm run prettify` before your `git commit`. -Codes will be formatted. +1. When you edit the code, please run `npm run prettify` to format your code before `git commit`. +2. In the description of your pull request, please include: + * Operating System version on which you tested + * Mattermost server version on which you tested + * New or updated unit tests for your changes +3. Please complete the [Mattermost CLA](http://www.mattermost.org/mattermost-contributor-agreement/) prior to submitting a PR. -Then, such as following points are helpful. - -* Tested OS -* Tested Mattermost version -* Test codes for your changes