Adds explaining the rationale behind your PR as a requirement in contributing.md (#22174)

* Use your words

* Updates the template

* Make it hidden
This commit is contained in:
KorPhaeron
2016-12-15 12:44:23 +13:00
committed by oranges
parent a40b3eede7
commit 10cc9294b4
2 changed files with 4 additions and 0 deletions
+2
View File
@@ -287,6 +287,8 @@ There is no strict process when it comes to merging pull requests, pull requests
* If your pull request is accepted, the code you add no longer belongs exclusively to you but to everyone; everyone is free to work on it, but you are also free to object to any changes being made, which will be noted by a Project Lead or Project Manager. It is a shame this has to be explicitly said, but there have been cases where this would've saved some trouble.
* Please explain why you are submitting the pull request, and how you think your change will be beneficial to the game. Failure to do so will be grounds for rejecting the PR.
## Banned content
Do not add any of the following in a Pull Request or risk getting the PR closed:
* National Socialist Party of Germany content, National Socialist Party of Germany related content, or National Socialist Party of Germany references
+2
View File
@@ -16,3 +16,5 @@ imagedel: deleted some icons and images
spellcheck: fixed a few typos
experiment: added an experimental thingy
/:cl:
[]: # (Please add a short description of why you think these changes would benefit the game. If you can't justify it in words, it might not be worth adding:)