Skip to content

Commit a4ed8d9

Browse files
authored
Merge pull request #8082 from cocobennett/improve-docs
Improve documentation on contributing to documentation
2 parents f7be0f1 + 5f477e4 commit a4ed8d9

File tree

3 files changed

+53
-7
lines changed

3 files changed

+53
-7
lines changed

docs/development/docs.rst

+52-7
Original file line numberDiff line numberDiff line change
@@ -5,32 +5,77 @@ As one might expect,
55
the documentation for Read the Docs is built using Sphinx and hosted on Read the Docs.
66
The docs are kept in the ``docs/`` directory at the top of the source tree.
77

8-
.. TODO: expand this section explaining there the PR is automatically built and
9-
the author can visualize changes without installing anything on their system.
10-
However, if there is going to be periodic/bigger contributions, it may be a
11-
good idea to install the Sphinx requirements to build our docs.
8+
Contributing through the Github UI
9+
----------------------------------
10+
11+
If you're making small changes to the documentation,
12+
you can verify those changes through the documentation generated when you open a PR and can be accessed using the Github UI.
13+
14+
#. click the checkmark next to your commit and it will expand to have multiple options
15+
16+
#. right-click the "details" link next to the "docs/readthedocs.org:docs" item
17+
18+
.. image:: /img/details_link.png
19+
20+
#. navigate to the section of the documentation you worked on to verify your changes
21+
22+
Contributing from your local machine
23+
------------------------------------
24+
25+
.. note::
26+
27+
You must use python 3.6 to generate the documentation.
28+
29+
If you're making large changes to the documentation,
30+
you may want to verify those changes locally before pushing upstream.
31+
32+
#. clone the `readthedocs.org` repository:
33+
34+
.. prompt:: bash
35+
36+
git clone --recurse-submodules https://github.com/readthedocs/readthedocs.org/
37+
38+
#. install documentation requirements
39+
40+
.. prompt:: bash
41+
42+
cd readthedocs.org
43+
pip install -r requirements/testing.txt
44+
pip install -r requirements/docs.txt
45+
46+
#. build the documents
47+
48+
.. prompt:: bash
49+
50+
cd docs
51+
make livehtml
52+
53+
#. the documents will be available at http://127.0.0.1:4444/ and will rebuild each time you edit and save a file.
54+
55+
Guidelines
56+
----------
1257

1358
Please follow these guidelines when updating our docs.
1459
Let us know if you have any questions or something isn't clear.
1560

1661
The brand
17-
---------
62+
^^^^^^^^^
1863

1964
We are called **Read the Docs**.
2065
The *the* is not capitalized.
2166

2267
We do however use the acronym **RTD**.
2368

2469
Titles
25-
------
70+
^^^^^^
2671

2772
For page titles, or Heading1 as they are sometimes called, we use title-case.
2873

2974
If the page includes multiple sub-headings (H2, H3),
3075
we usually use sentence-case unless the titles include terminology that is supposed to be capitalized.
3176

3277
Content
33-
-------
78+
^^^^^^^
3479

3580
* Do not break the content across multiple lines at 80 characters,
3681
but rather break them on semantic meaning (e.g. periods or commas).

docs/img/details_link.png

40.3 KB
Loading

requirements/docs.txt

+1
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,7 @@ sphinx-prompt==1.4.0
2727
sphinx-notfound-page==0.6
2828
commonmark==0.9.1
2929
recommonmark==0.7.1
30+
sphinx-autobuild==2021.3.14
3031

3132
# Linting
3233
rstcheck==3.3.1

0 commit comments

Comments
 (0)