added a check list for pull request submissions

lsmith77 · lsmith77 · commit e92c57d87a3a · 2011-09-23T14:02:29.000+02:00
diff --git a/contributing/code/patches.rst b/contributing/code/patches.rst
@@ -4,6 +4,48 @@ Submitting a Patch
 Patches are the best way to provide a bug fix or to propose enhancements to
 Symfony2.
 
+Check List
+----------
+
+The purpose of the check list is to ensure that contributions may be reviewed
+with needless feedback loops to ensure that your contributions can be included
+into Symfony2 as quickly as possible.
+
+.. warning::
+
+    All pull requests should include the following template in the request
+    description:
+
+.. code-block:: plain
+
+    Bug fix: [yes|no]
+    Feature addition: [yes|no]
+    Backwards compatibility break: [yes|no]
+    Symfony2 tests pass: [yes|no]
+    Fixes the following tickets: [comma separated list of tickets fixed by the PR]
+    Description: [description of the change]
+
+An example submission could now look as follows:
+
+.. code-block:: plain
+
+    Bug fix: no
+    Feature addition: yes
+    Backwards compatibility break: no
+    Symfony2 tests pass: yes
+    Fixes the following tickets: -
+    Description: Add X to Y to make the world a brighter place
+
+Thank you for including the filled out template in your submission!
+
+.. tip::
+
+    All feature addition's should be send to the "master" branch, while all
+    bug fixes should be send to the oldest still active branch. Furthermore
+    submissions should never break backwards compatibility. Please
+    mark any submissions as "[WIP]" in the title in case the submission
+    is not yet complete (for example if the tests do not yet pass).
+
 Initial Setup
 -------------
 
