Skip to content

Commit c0091d4

Browse files
committed
Merge pull request #2150 from symfony/docs-contributing
Adding an official format to be used for documentation pull requests
2 parents 1d3a214 + 3d45abf commit c0091d4

File tree

3 files changed

+40
-6
lines changed

3 files changed

+40
-6
lines changed

CONTRIBUTING.md

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,4 +2,6 @@ Contributing
22
------------
33

44
We love contributors! For more information on how you can contribute to the
5-
Symfony documentation, please read [Contributing to the Documentation](http://symfony.com/doc/current/contributing/documentation/overview.html)
5+
Symfony documentation, please read [Contributing to the Documentation](http://symfony.com/doc/current/contributing/documentation/overview.html)
6+
and notice the [Pull Request Format](http://symfony.com/doc/current/contributing/documentation/overview.html#pull-request-format)
7+
that helps us merge your pull requests faster!

contributing/code/patches.rst

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -264,7 +264,7 @@ pull request message, like in:
264264
[Yaml] fixed something
265265
[Form] [Validator] [FrameworkBundle] added something
266266
267-
The pull request description must include the following check list at the top
267+
The pull request description must include the following checklist at the top
268268
to ensure that contributions may be reviewed without needless feedback
269269
loops and that your contributions can be included into Symfony2 as quickly as
270270
possible:

contributing/documentation/overview.rst

Lines changed: 36 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -46,8 +46,12 @@ Next, create a dedicated branch for your changes (for organization):
4646
4747
You can now make your changes directly to this branch and commit them. When
4848
you're done, push this branch to *your* GitHub fork and initiate a pull request.
49-
The pull request will be between your ``improving_foo_and_bar`` branch and
50-
the ``symfony-docs`` ``master`` branch.
49+
50+
Creating a Pull Request
51+
~~~~~~~~~~~~~~~~~~~~~~~
52+
53+
Following the example, the pull request will default to be between your
54+
``improving_foo_and_bar`` branch and the ``symfony-docs`` ``master`` branch.
5155

5256
.. image:: /images/docs-pull-request.png
5357
:align: center
@@ -60,8 +64,8 @@ the base branch to be 2.0 on the preview page:
6064

6165
.. note::
6266

63-
All changes made to the 2.0 branch will be merged into 2.1 which in turn will be
64-
merged into the master branch for the next release on a weekly basis.
67+
All changes made to a branch (e.g. 2.0) will be merged up to each "newer"
68+
branch (e.g. 2.1, master, etc) for the next release on a weekly basis.
6569

6670
GitHub covers the topic of `pull requests`_ in detail.
6771

@@ -70,6 +74,34 @@ GitHub covers the topic of `pull requests`_ in detail.
7074
The Symfony2 documentation is licensed under a Creative Commons
7175
Attribution-Share Alike 3.0 Unported :doc:`License <license>`.
7276

77+
Pull Request Format
78+
~~~~~~~~~~~~~~~~~~~
79+
80+
Unless you're fixing some minor typos, the pull request description must**
81+
include the following checklist to ensure that contributions may be reviewed
82+
without needless feedback loops and that your contributions can be included
83+
into the documentation as quickly as possible:
84+
85+
.. code-block:: text
86+
87+
| Q | A
88+
| ------------- | ---
89+
| Doc fix? | [yes|no]
90+
| New docs? | [yes|no] (PR # on symfony/symfony if applicable)
91+
| Applies to | [Symfony version numbers this applies to]
92+
| Fixed tickets | [comma separated list of tickets fixed by the PR]
93+
94+
An example submission could now look as follows:
95+
96+
.. code-block:: text
97+
98+
| Q | A
99+
| ------------- | ---
100+
| Doc fix? | yes
101+
| New docs? | yes (symfony/symfony#2500)
102+
| Applies to | all (or 2.1+)
103+
| Fixed tickets | #1075
104+
73105
.. tip::
74106

75107
Please be patient. It can take from 15 minutes to several days for your changes

0 commit comments

Comments
 (0)