Tool: PEP8 Style Checker
Ignored Files:
    docs/codebase/generating-documentation.rst
    docs/codebase/index.rst



Tool: Pyflakes
Ignored Files:
    docs/codebase/generating-documentation.rst
    docs/codebase/index.rst

<p>Please write a summary that describes the general issue and fill out the testing done. The bug number should also go in the bugs field.</p>

<p>You don't need to include files that were added or modified in your change descriptions. You may want to read <a href="https://www.reviewboard.org/docs/codebase/dev/writing-good-descriptions/">Writing Good Descriptions</a>.</p>

This file should be wrapped at 79 characters.

easy_install...
I prefer pip, though.

This isn't particular to installing documentation. Also, apt-get will only work for users on Debian based systems.

"browse to..."
We should put the path in double backquotes so it appears as monospaced text formated as code.

Some of the steps here (like the Sphinx installation) are no longer going to be necessary once https://reviews.reviewboard.org/r/7629/ lands. You'll want to update the steps to refer to the Getting Started guide for those parts.

Tool: Pyflakes
Ignored Files:
    docs/codebase/generating-documentation.rst
    docs/codebase/index.rst



Tool: PEP8 Style Checker
Ignored Files:
    docs/codebase/generating-documentation.rst
    docs/codebase/index.rst

The summary here "Easyfix: 3703" isn't particularly helpful. It forces someone reading the git log to go to the bug tracker to find out what happened. We also would like more detail in the description, and some details on how you tested this.
Please see https://www.reviewboard.org/docs/codebase/dev/writing-good-descriptions/

We don't need this split up into tiny sections. This can all go into one section (e.g., Building documentation).

Use double backquotes.
I would also not use "get into" for changing directory. I would say something like

The documentation resides in the ``/reviewboard/docs/manual`` directory.

Instead of "simply type the following" how about "install Sphinx via"

This can be replaced with something like "To generate the documentation, run the following:"

This doesn't really need to be here.

Tool: PEP8 Style Checker
Ignored Files:
    docs/codebase/generating-documentation.rst
    docs/codebase/index.rst



Tool: Pyflakes
Ignored Files:
    docs/codebase/generating-documentation.rst
    docs/codebase/index.rst

Tool: PEP8 Style Checker
Ignored Files:
    docs/codebase/generating-documentation.rst
    docs/codebase/index.rst



Tool: Pyflakes
Ignored Files:
    docs/codebase/generating-documentation.rst
    docs/codebase/index.rst

This should be double backticks, e.g.
``/reviewboard/docs/manual``

Likewise, this too.

Tool: PEP8 Style Checker
Ignored Files:
    docs/codebase/generating-documentation.rst
    docs/codebase/index.rst



Tool: Pyflakes
Ignored Files:
    docs/codebase/generating-documentation.rst
    docs/codebase/index.rst

Please wrap this to 80 characters.

Tool: PEP8 Style Checker
Ignored Files:
    docs/codebase/generating-documentation.rst
    docs/codebase/index.rst



Tool: Pyflakes
Ignored Files:
    docs/codebase/generating-documentation.rst
    docs/codebase/index.rst

Ship It!