Care to upload the images?

This is in beanbag-docutils as beanbag_docutils.sphinx.ext.retina_images. We should prefer that over this.

This happened to me too. Looks like its expanding \titleref to a TAB and itleref. Silly Sphinx.

Aren't code blocks in reST supposed to line up with the c in code-block?

We don't actually supply the message broker, do we? Isn't that RabbitMQ, etc?

Ship It!

It'll be nice to have a manual to link to :)

Something that stood out strongly to me is that information on tools are scattered across too many pages. If I want to know how to install PMD, I go one place. If I want to know how to configure it, I go somewhere else. If I want to learn more about what it is, a third place.
I think instead, we should take the approach that repositories/hosting services now have in the Review Board docs. We should make reviewbot/tools/ a directory and have clang.rst, pmd.rst, etc. in there. These should be complete documents on installing, configuring, and using these tools. That way it's really easy to see all the information needed about each tool without navigating different pages. As we expand our supported tools, this will help keep things far more manageable.

The URL should be https://www.reviewboard.org/docs/manual/latest/
Otherwise we'll have 2 redirects for this.

Can you add a reference anchor for this page, so we'll be able to more easily link to it in the future? I've found it really helps to always have one of these for each page.

Maybe provide an example of what this would look like if running locally.

Both "command line" and "command-line" are valid terms, but we should probably standardize on one. Most of our docs say "command line," but some say "command-line." I did some searches to see which is more popular out there, and it seems it's "command line". It's also far more common in search terms:
https://trends.google.com/trends/explore?q=%22command-line%22,%22command%20line%22
It seems that Apple and newer Microsoft docs are using "command line" over "command-line" as well. I wish I could find some good info on best practices here from these companies.

Can we link to PMD? If you don't already know what it is, this paragraph is very confusing, and sounds like a requirement for Review Bot. We should have a brief summary of what PMD is before going into configuration for it.

Let's link this.

Can we add a reference anchor here too?

Let's just go ahead and link to this file on GitHub.

Can we say "the Review Bot extension for Review Board" instead? I had to read this a couple times before I parsed it right.

Inconsistencies with the number of blank lines here (and below, wiht .. image).

Should be httpd, not http.
The standard way of working with services these days (I think this has been true for a few years now) is sudo service httpd reload.

No need for the backticks on these.

Too many blank lines.

Instead of linking to this, we should add an intersphinx for celery in the Sphinx config file, naming it celery. Then we can do:
Celery's `supported brokers <celery:brokers>`_

above.

Should be only one blank line.
This is also in the wrong section.

We should link this.

Too many blank lines.

Here, too.

Can you add a reference anchor here?

We should use "current" instead of "latest". They use "latest" as in "bleeding edge," not "current release."

No need for backticks. Same with others below.

No backticks. They're never needed on .. lines.

Is two trailing slashes correct?

There's 2 spaces here between "and" and "click".

"Celery" ?

No need for backticks.

No need for backticks.

No need for backticks.

Too many blank lines.

No need for backticks.

No need for backticks.

No need for backticks.

Sphinx has a handy way to reference PEPs. You can just do:
:pep:`8`

"installed"

No need for backticks.

Ship It!