Skip to content

Commit 279527c

Browse files
committed
[IMP] contributing/documentation: update documentation guidelines
task-3709518 closes #11023 X-original-commit: 9225abb Signed-off-by: Audrey Vandromme (auva) <auva@odoo.com>
1 parent c6208a5 commit 279527c

24 files changed

+1648
-1494
lines changed

content/contributing/check_mergeability_status.rst

Lines changed: 0 additions & 1 deletion
This file was deleted.

content/contributing/configure_git_authorship.rst

Lines changed: 0 additions & 7 deletions
This file was deleted.

content/contributing/configure_github_account.rst

Lines changed: 0 additions & 2 deletions
This file was deleted.

content/contributing/create_github_account.rst

Lines changed: 0 additions & 3 deletions
This file was deleted.

content/contributing/development.rst

Lines changed: 18 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -28,15 +28,25 @@ The instructions below help you prepare your environment for making local change
2828
and then push them to GitHub. Skip this section and go to
2929
:ref:`contributing/development/first-contribution` if you have already completed this step.
3030

31-
#. .. include:: create_github_account.rst
32-
#. .. include:: configure_github_account.rst
31+
#. First, you need to `create a GitHub account <https://github.com/join>`_. Odoo uses GitHub to
32+
manage the source code of its products, and this is where you will make your changes and submit
33+
them for review.
34+
#. `Generate a new SSH key and register it on your GitHub account
35+
<https://docs.github.com/en/authentication/connecting-to-github-with-ssh>`_.
3336
#. Go to `github.com/odoo/odoo <https://github.com/odoo/odoo>`_ and click on the :guilabel:`Fork`
3437
button in the top right corner to create a fork (:dfn:`your own copy`) of the repository on your
3538
account. Do the same with `github.com/odoo/enterprise <https://github.com/odoo/enterprise>`_ if
3639
you have access to it. This creates a copy of the codebase to which you can make changes without
3740
affecting the main codebase. Skip this step if you work at Odoo.
3841
#. .. include:: install_git.rst
39-
#. .. include:: configure_git_authorship.rst
42+
#. Configure Git to identify yourself as the author of your future contributions. Enter the same
43+
email address you used to register on GitHub.
44+
45+
.. code-block:: console
46+
47+
$ git config --global user.name "Your Name"
48+
$ git config --global user.email "youremail@example.com"
49+
4050
#. :doc:`Install Odoo from the sources <../administration/on_premise/source>`. Make sure to fetch
4151
the sources through Git with SSH.
4252
#. Configure Git to push changes to your fork(s) rather than to the main codebase. If you work at
@@ -155,7 +165,10 @@ navigate to the directory where you installed Odoo from sources and follow the g
155165
#. Tick the :guilabel:`Allow edits from maintainer` checkbox. Skip this step if you work at Odoo.
156166
#. Complete the description and click on the :guilabel:`Create pull request` button again.
157167

158-
#. .. include:: check_mergeability_status.rst
159-
#. .. include:: handle_reviews.rst
168+
#. At the bottom of the page, check the mergeability status and address any issues.
169+
#. As soon as your :abbr:`PR (Pull Request)` is ready for merging, a member of the Odoo team
170+
is automatically assigned for review. If the reviewer has questions or remarks, they will
171+
post them as comments and you will be notified by email. Those comments must be resolved
172+
for the contribution to go forward.
160173
#. Once your changes are approved, the review merges them and they become available for all Odoo
161174
users after the next code update!

content/contributing/documentation.rst

Lines changed: 57 additions & 51 deletions
Original file line numberDiff line numberDiff line change
@@ -8,54 +8,40 @@ Documentation
88
:titlesonly:
99

1010
documentation/content_guidelines
11-
documentation/rst_cheat_sheet
1211
documentation/rst_guidelines
1312

14-
This introductory guide will help you acquire the tools and knowledge you need to write
15-
documentation, whether you plan to make a minor content change or document an application from
16-
scratch.
13+
This introductory guide will help you acquire the tools and knowledge needed to contribute to the
14+
documentation.
15+
16+
Read the :ref:`introduction to the reStructuredText language <contributing/documentation/rst-intro>`
17+
if you are not familiar with it. Then, there are two courses of action to start contributing to the
18+
documentation:
19+
20+
- **For minor changes**, such as adding a paragraph or fixing a typo, we recommend **using the
21+
GitHub interface**. This is the easiest and fastest way to submit changes, and it is suitable for
22+
non-technical people. Jump directly to the :ref:`contributing/documentation/first-contribution`
23+
section to get started.
24+
- **For more complex changes**, such as adding a new page, it is necessary to **use Git** and work
25+
from a local copy of the documentation. Follow the instructions in the
26+
:ref:`contributing/documentation/setup` section first to prepare your environment.
1727

1828
.. seealso::
1929
:doc:`Discover other ways to contribute to Odoo <../contributing>`
2030

21-
Read the :ref:`introduction to the reStructuredText language <contributing/documentation/rst-intro>`
22-
if you are not familiar with it. Then, you have two courses of action to start contributing to the
23-
documentation, depending on whether you want to propose minor changes to existing content or you
24-
instead want to work on significant changes to new and existing content.
25-
26-
- **For minor changes**, for example, adding a paragraph or fixing a typo, we recommend **using the
27-
GitHub interface**. This is the easiest and fastest way to submit your changes, and it is suitable
28-
for non-technical people. Jump directly to the
29-
:ref:`contributing/documentation/first-contribution` section to get started.
30-
- **For more complex changes**, it is necessary to **use Git** and work from a local copy of the
31-
documentation. Follow the instructions in the :ref:`contributing/documentation/setup` section to
32-
first prepare your environment.
33-
3431
.. _contributing/documentation/rst-intro:
3532

3633
reStructuredText (RST)
3734
======================
3835

3936
The documentation is written in **reStructuredText** (RST), a `lightweight markup language
4037
<https://en.wikipedia.org/wiki/Lightweight_markup_language>`_ consisting of regular text augmented
41-
with markup, which allows including headings, images, notes, and so on. This might seem a bit
42-
abstract, but there is no need to worry; :abbr:`RST (reStructuredText)` is not hard to learn,
43-
especially if you intend to make minor changes to the content.
44-
45-
If you need to learn about a specific markup, head over to our :doc:`cheat sheet for RST
46-
<documentation/rst_cheat_sheet>`; it contains all the information you should ever need for the
47-
documentation of Odoo.
38+
with markup, which allows including headings, images, notes, and so on. :abbr:`RST
39+
(reStructuredText)` is easy to use, even if you are not familiar with it.
4840

4941
.. important::
50-
We kindly ask you to observe a set of :doc:`content <documentation/content_guidelines>` and
42+
Be mindful of our :doc:`content <documentation/content_guidelines>` and
5143
:doc:`RST <documentation/rst_guidelines>` guidelines as you write documentation. This ensures
52-
that you stay consistent with the rest of the documentation and facilitates the approval of your
53-
content changes as the Odoo team reviews them.
54-
55-
.. seealso::
56-
- :doc:`documentation/content_guidelines`
57-
- :doc:`documentation/rst_cheat_sheet`
58-
- :doc:`documentation/rst_guidelines`
44+
that the documentation stays consistent and facilitates the approval of changes by the Odoo team.
5945

6046
.. _contributing/documentation/setup:
6147

@@ -67,14 +53,23 @@ documentation and then push them to GitHub. Skip this section and go to
6753
:ref:`contributing/documentation/first-contribution` if you have already completed this step or want
6854
to make changes from the GitHub interface.
6955

70-
#. .. include:: create_github_account.rst
71-
#. .. include:: configure_github_account.rst
56+
#. First, `create a GitHub account <https://github.com/join>`_. Odoo uses GitHub to manage the
57+
source code of its products, and this is where you will submit your changes.
58+
#. `Generate a new SSH key and register it on your GitHub account
59+
<https://docs.github.com/en/authentication/connecting-to-github-with-ssh>`_.
7260
#. Go to `github.com/odoo/documentation <https://github.com/odoo/documentation>`_ and click on the
7361
:guilabel:`Fork` button in the top right corner to create a fork (:dfn:`your own copy`) of the
7462
repository on your account. This creates a copy of the codebase to which you can make changes
7563
without affecting the main codebase. Skip this step if you work at Odoo.
7664
#. .. include:: install_git.rst
77-
#. .. include:: configure_git_authorship.rst
65+
#. Configure Git to identify yourself as the author of your future contributions. Enter the same
66+
email address you used to register on GitHub.
67+
68+
.. code-block:: console
69+
70+
$ git config --global user.name "Your Name"
71+
$ git config --global user.email "youremail@example.com"
72+
7873
#. Clone the sources with Git and navigate into the local repository.
7974

8075
.. code-block:: console
@@ -109,7 +104,7 @@ to make changes from the GitHub interface.
109104
$ git config commit.template %CD%\commit_template.txt
110105
111106
#. Install the latest release of `Python <https://wiki.python.org/moin/BeginnersGuide/Download>`_
112-
and `pip <https://pip.pypa.io/en/stable/installation/>`_ on your machine.
107+
and `pip <https://pip.pypa.io/en/stable/installation/>`_.
113108
#. Install the Python dependencies of the documentation with pip.
114109

115110
.. code-block:: console
@@ -149,18 +144,20 @@ to make changes from the GitHub interface.
149144
<https://www.technewstoday.com/install-and-use-make-in-windows>`_.
150145

151146
#. `Install pngquant <https://pngquant.org/>`_.
152-
#. That's it! You are ready to :ref:`make your first contribution
147+
#. You are now ready to :ref:`make your first contribution
153148
<contributing/documentation/first-contribution>` with Git.
154149

155150
.. _contributing/documentation/first-contribution:
156151

157-
Make your first contribution
158-
============================
152+
Contributing to the documentation
153+
=================================
159154

160155
.. tabs::
161156

162157
.. tab:: Contribute from the GitHub interface
163-
#. .. include:: create_github_account.rst
158+
159+
#. First, `create a GitHub account <https://github.com/join>`_. Odoo uses GitHub to manage the
160+
source code of its products, and this is where you will submit your changes.
164161
#. Verify that you are browsing the documentation in the version that you intend to change.
165162
The version can be selected from the dropdown in the top menu.
166163
#. Head to the page that you want to change and click on the :guilabel:`Edit on GitHub` button
@@ -194,9 +191,14 @@ Make your first contribution
194191
Odoo.
195192
#. Review the summary that you wrote about your changes and click on the :guilabel:`Create
196193
pull request` button again.
197-
#. .. include:: check_mergeability_status.rst
198-
#. .. include:: handle_reviews.rst
199-
#. .. include:: documentation/changes_approved.rst
194+
#. At the bottom of the page, check the mergeability status and address any issues.
195+
#. As soon as your :abbr:`PR (Pull Request)` is ready for merging, a member of the Odoo team
196+
is automatically assigned for review. If the reviewer has questions or remarks, they will
197+
post them as comments and you will be notified by email. Those comments must be resolved
198+
for the contribution to go forward.
199+
200+
#. Once your changes are approved, the reviewer merges them and they appear online the next
201+
day.
200202

201203
.. tab:: Contribute with Git
202204

@@ -230,7 +232,7 @@ Make your first contribution
230232
#. Make the desired changes while taking care of following the :doc:`content
231233
<documentation/content_guidelines>` and :doc:`RST <documentation/rst_guidelines>`
232234
guidelines.
233-
#. Compress all PNG images that you added or modified.
235+
#. Compress all PNG images that were added or modified.
234236

235237
.. code-block:: console
236238
@@ -239,9 +241,9 @@ Make your first contribution
239241
240242
#. Write a `redirect rule
241243
<https://github.com/odoo/documentation/tree/{BRANCH}/redirects/MANUAL.md>`_ for every RST
242-
file that your renamed.
243-
#. Build the documentation with :command:`make`. Then, open :file:`_build/index.html` in your
244-
web browser to browse the documentation with your changes.
244+
file that were renamed.
245+
#. Build the documentation with :command:`make`. Then, open :file:`_build/index.html` in a web
246+
browser to browse the documentation with your changes.
245247

246248
.. tip::
247249
Use :command:`make help` to learn about other useful commands.
@@ -254,7 +256,7 @@ Make your first contribution
254256
$ git add .
255257
$ git commit
256258
257-
#. Push your change to your fork, for which we added the remote alias `dev`.
259+
#. Push your changes to your fork, for which we added the remote alias `dev`.
258260

259261
.. example::
260262

@@ -285,6 +287,10 @@ Make your first contribution
285287
Odoo.
286288
#. Complete the description and click on the :guilabel:`Create pull request` button again.
287289

288-
#. .. include:: check_mergeability_status.rst
289-
#. .. include:: handle_reviews.rst
290-
#. .. include:: documentation/changes_approved.rst
290+
#. At the bottom of the page, check the mergeability status and address any issues.
291+
#. As soon as your :abbr:`PR (Pull Request)` is ready for merging, a member of the Odoo team
292+
is automatically assigned for review. If the reviewer has questions or remarks, they will
293+
post them as comments and you will be notified by email. Those comments must be resolved
294+
for the contribution to go forward.
295+
#. Once your changes are approved, the reviewer merges them and they appear online the next
296+
day.

content/contributing/documentation/changes_approved.rst

Lines changed: 0 additions & 1 deletion
This file was deleted.

0 commit comments

Comments
 (0)