Skip to content

Commit d987dee

Browse files
committed
[Contributing] Several tweaks
1 parent 1972757 commit d987dee

File tree

4 files changed

+52
-19
lines changed

4 files changed

+52
-19
lines changed

Diff for: contributing/documentation/format.rst

+2-1
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,7 @@ reStructuredText
1111
reStructuredText is a plaintext markup syntax similar to Markdown, but much
1212
stricter with its syntax. If you are new to reStructuredText, take some time to
1313
familiarize with this format by reading the existing `Symfony documentation`_
14+
in raw format.
1415

1516
If you want to learn more about this format, check out the `reStructuredText Primer`_
1617
tutorial and the `reStructuredText Reference`_.
@@ -182,7 +183,7 @@ Symfony, you should precede your description of the change with a
182183
You can also ask a question and hide the response. This is particularly [...]
183184
184185
If you're documenting a behavior change, it may be helpful to *briefly* describe
185-
how the behavior has changed.
186+
how the behavior has changed:
186187

187188
.. code-block:: rst
188189

Diff for: contributing/documentation/overview.rst

+24-9
Original file line numberDiff line numberDiff line change
@@ -83,15 +83,15 @@ and click on the ``Pull Requests`` link located in the sidebar.
8383

8484
Then, click on the big ``New pull request`` button. As GitHub cannot guess the
8585
exact changes that you want to propose, select the appropriate branches where
86-
changes should be applied:º
86+
changes should be applied:
8787

8888
.. image:: /images/contributing/docs-pull-request-change-base.png
8989
:align: center
9090

91-
In this example, the **base repository** should be ``symfony/symfony-docs`` and
92-
the **base branch** should be the ``2.3``, which is the branch that you selected
93-
to base your changes on. The **compare repository** should be your forked copy
94-
of ``symfony-docs`` and the **compare branch** should be ``improve_install_chapter``,
91+
In this example, the **base fork** should be ``symfony/symfony-docs`` and
92+
the **base** branch should be the ``2.3``, which is the branch that you selected
93+
to base your changes on. The **head fork** should be your forked copy
94+
of ``symfony-docs`` and the **compare** branch should be ``improve_install_chapter``,
9595
which is the name of the branch you created and where you made your changes.
9696

9797
.. _pull-request-format:
@@ -136,7 +136,7 @@ changes and push them:
136136
137137
# ... do your changes
138138
139-
$ git push
139+
$ git push origin improve_install_chapter
140140
141141
**Step 10.** After your pull request is eventually accepted and merged in the Symfony
142142
documentation, you will be included in the `Symfony Documentation Contributors`_
@@ -188,7 +188,7 @@ section:
188188
# submit the changes to your forked repository
189189
$ git add xxx.rst # (optional) only if this is a new content
190190
$ git commit xxx.rst
191-
$ git push
191+
$ git push origin my_changes
192192
193193
# go to GitHub and create the Pull Request
194194
#
@@ -230,7 +230,7 @@ steps to contribute to the Symfony documentation, which you can use as a
230230
# add and commit your changes
231231
$ git add xxx.rst # (optional) only if this is a new content
232232
$ git commit xxx.rst
233-
$ git push
233+
$ git push origin my_changes
234234
235235
# go to GitHub and create the Pull Request
236236
#
@@ -244,10 +244,25 @@ steps to contribute to the Symfony documentation, which you can use as a
244244
245245
# (optional) make the changes requested by reviewers and commit them
246246
$ git commit xxx.rst
247-
$ git push
247+
$ git push origin my_changes
248248
249249
You guessed right: after all this hard work, it's **time to celebrate again!**
250250

251+
Minor Changes (e.g. Typos)
252+
--------------------------
253+
254+
You may find just a typo and want to fix it. Due to GitHub's functional
255+
frontend, it is quite simple to create Pull Requests right in your
256+
browser while reading the docs on **symfony.com**. To do this, just click
257+
the ``edit this page`` button on the upper right corner. Beforehand
258+
please switch to the right branch as mentioned before. Now you are able
259+
to edit the content and describe your changes within the GitHub
260+
frontend. When your work is done, click ``propose file change`` to
261+
create a commit and, in case it is your first contribution, also your
262+
fork. A new branch is created automatically in order to provide a base
263+
for your Pull Request. Then fill out the form to create the Pull Request
264+
as described above.
265+
251266
Frequently Asked Questions
252267
--------------------------
253268

Diff for: contributing/documentation/standards.rst

+25-8
Original file line numberDiff line numberDiff line change
@@ -7,8 +7,14 @@ look and feel familiar, you should follow these standards.
77
Sphinx
88
------
99

10-
* The following characters are chosen for different heading levels: level 1
11-
is ``=``, level 2 ``-``, level 3 ``~``, level 4 ``.`` and level 5 ``"``;
10+
* The following characters are chosen for different heading levels:
11+
12+
* level 1 ``=``;
13+
* level 2 ``-``;
14+
* level 3 ``~``;
15+
* level 4 ``.``;
16+
* level 5 ``"``;
17+
1218
* Each line should break approximately after the first word that crosses the
1319
72nd character (so most lines end up being 72-78 characters);
1420
* The ``::`` shorthand is *preferred* over ``.. code-block:: php`` to begin a PHP
@@ -53,13 +59,22 @@ Code Examples
5359
* To avoid horizontal scrolling on code blocks, we prefer to break a line
5460
correctly if it crosses the 85th character;
5561
* When you fold one or more lines of code, place ``...`` in a comment at the point
56-
of the fold. These comments are: ``// ...`` (php), ``# ...`` (yaml/bash), ``{# ... #}``
57-
(twig), ``<!-- ... -->`` (xml/html), ``; ...`` (ini), ``...`` (text);
62+
of the fold. These comments are:
63+
64+
* ``// ...`` (php);
65+
* ``# ...`` (yaml/bash);
66+
* ``{# ... #}`` (twig);
67+
* ``<!-- ... -->`` (xml/html);
68+
* ``; ...`` (ini);
69+
* ``...`` (text);
70+
5871
* When you fold a part of a line, e.g. a variable value, put ``...`` (without comment)
5972
at the place of the fold;
6073
* Description of the folded code: (optional)
61-
If you fold several lines: the description of the fold can be placed after the ``...``
62-
If you fold only part of a line: the description can be placed before the line;
74+
75+
* If you fold several lines: the description of the fold can be placed after the ``...``;
76+
* If you fold only part of a line: the description can be placed before the line;
77+
6378
* If useful to the reader, a PHP code example should start with the namespace
6479
declaration;
6580
* When referencing classes, be sure to show the ``use`` statements at the
@@ -77,8 +92,9 @@ Configuration examples should show all supported formats using
7792
:ref:`configuration blocks <docs-configuration-blocks>`. The supported formats
7893
(and their orders) are:
7994

80-
* **Configuration** (including services and routing): YAML, XML, PHP
81-
* **Validation**: YAML, Annotations, XML, PHP
95+
* **Configuration** (including services): YAML, XML, PHP
96+
* **Routing**: Annotations, YAML, XML, PHP
97+
* **Validation**: Annotations, YAML, XML, PHP
8298
* **Doctrine Mapping**: Annotations, YAML, XML, PHP
8399
* **Translation**: XML, YAML, PHP
84100

@@ -151,6 +167,7 @@ English Language Standards
151167
* **Gender-neutral language**: when referencing a hypothetical person, such as
152168
*"a user with a session cookie"*, use gender-neutral pronouns (they/their/them).
153169
For example, instead of:
170+
154171
* he or she, use they
155172
* him or her, use them
156173
* his or her, use their

Diff for: contributing/documentation/translations.rst

+1-1
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ in the translation process.
66

77
.. note::
88

9-
Symfony Project officially discourages starting new translations for the
9+
Symfony Project officially discourages from starting new translations for the
1010
documentation. As a matter of fact, there is `an ongoing discussion`_ in
1111
the community about the benefits and drawbacks of community driven translations.
1212

0 commit comments

Comments
 (0)