Merge pull request #1334 from CapnBry/z-grid-dest

[clinton/Smoothieware.git] / PULL_REQUEST_TEMPLATE.md

diff --git a/PULL_REQUEST_TEMPLATE.md b/PULL_REQUEST_TEMPLATE.md
index 3f670fc..a7535bf 100644 (file)
--- a/PULL_REQUEST_TEMPLATE.md
+++ b/PULL_REQUEST_TEMPLATE.md
@@ -3,13 +3,13 @@ In order for a Pull request to be accepted it must meet certain standards and gu
 In addition to following the [Coding-Standards](http://smoothieware.org/coding-standards) it is suggested the following be adopted.
 
 * There must be a detailed description of the change, why it was added and what the use case is, and a link to the documentation to use it in the wiki if applicable.
+* Do not commit changes to the FirmwareBin directory or any binary files in the repository. (this is hard to reverse once done)
 * Only one feature per pull request, if you change multiple files with unrelated changes it will be rejected.
 * Do not make gratuitous changes. If your coding style is different from the one adopted by most of the Smoothie codebase, do not make changes to suit your style. Peoples styles differ, and the new code you write may reflect your coding style, that is ok, but changing existing code to match your style is not ok. Doing so will delay the acceptance of your code, or have it rejected entirely.
 * Maintain backwards compatibility. Do not make changes that will break other peoples systems, it is not acceptable to force everyone to change their config files to accommodate your change. If necessary add new configs to enable your change if it is breaking backward compatibility.
 * Do not make changes that are specific to your setup. Smoothie needs to work in many different environments.
 * Do not make changes that customize messages specific to your company or implementation. You are welcome to do this in your own fork and binary, it is not ok to try to merge that into the upstream repository.
 * Test your changes is as many different configurations as you can, eg different arm solutions. In the Pull request state what tests have been done and what tests need to be done.
-* Do not commit changes to the FirmwareBin directory or any binary files in the repository.
 * Do not reformat the Config Samples to suit your style.
 * Add examples of your config entries to the ConfigSamples/Snippets folder, only change existing config samples if absolutely necessary
 * If you must correct existing comments (like correct spelling mistakes or typos) then submit them as separate pull requests