Tool: Pyflakes
Processed Files:
    setup.py
    rbtools/commands/__init__.py



Tool: PEP8 Style Checker
Processed Files:
    setup.py
    rbtools/commands/__init__.py

Col: 25
 E241 multiple spaces after ':'

Col: 24
 E241 multiple spaces after ':'

Col: 27
 E241 multiple spaces after ':'

Col: 25
 E241 multiple spaces after ':'

Tool: PEP8 Style Checker
Processed Files:
    setup.py
    rbtools/commands/__init__.py



Tool: Pyflakes
Processed Files:
    setup.py
    rbtools/commands/__init__.py

We usually use single-quotes around our strings instead of double-quotes (see line 577), and it's generally recommended (although not always enforced) that we use trailing commas if we are listing a different argument on each line like this (see line 588). Also, see my below comment.

Consider replacing this with handler.setFormatter(formatter) and then moving the colour checking up to where you initialize formatter. So we could replace the code starting at line 576 with:

if self.options.nocolor:
    formatter = logging.Formatter('%(levelname)s: %(message)s')
else:
    formatter = ColoredFormatter(
        '%(log_color)s%(levelname)-8s%(reset)s %(message)s',
        datefmt=None,
        reset=True,
        log_colors={
            'DEBUG': 'cyan',
            'INFO': 'green',
            'WARNING': 'yellow',
            'ERROR': 'red,bg_black',
            'CRITICAL': 'red,bg_white',
        },
        secondary_log_colors={},
        style='%',
    )


Also note how I used single quotes for the string instead of double quotes, as well as how I used a trailing comma for style='%',.

Naming-wise, this should be NO_COLOR.
However, it's best to have flags be more like "enabled" than "disabled". So, I'd prefer having a COLOR=False than NO_COLOR=True.
You can do this by changing the name, changing the action to 'store_false', and changing the default.

Trailing period.
You don't need to say "This feature." Instead, how about simply "Disables color output."

I just know we're going to hit some situation where this looks bad on someone's system. How about allowing each of these to be customized through .reviewboardrc, with these as defaults?

Blank line between these.

And these.

Can you split this across multiple lines?

Tool: Pyflakes
Processed Files:
    setup.py
    rbtools/commands/__init__.py



Tool: PEP8 Style Checker
Processed Files:
    setup.py
    rbtools/commands/__init__.py

This can still be all on one line. We should also keep it in alphabetical order.
If we want to wrap it, let's do so like:

install_requires = [
    'colorlog',
    'six>=1.8.0'
]

At this point, what's left to keep this as "WIP"?
I'm assuming that there are more parts to this project (like colorizing output of rbt diff and rbt status), but I'd also assume those would be separate review requests.

These can probably go away now, right?

Tool: Pyflakes
Processed Files:
    setup.py
    rbtools/commands/__init__.py



Tool: PEP8 Style Checker
Processed Files:
    setup.py
    rbtools/commands/__init__.py

Col: 17
 E128 continuation line under-indented for visual indent

Col: 17
 E128 continuation line under-indented for visual indent

Col: 17
 E128 continuation line under-indented for visual indent

Col: 21
 E127 continuation line over-indented for visual indent

Col: 21
 E127 continuation line over-indented for visual indent

Col: 21
 E127 continuation line over-indented for visual indent

Col: 21
 E127 continuation line over-indented for visual indent

Col: 21
 E127 continuation line over-indented for visual indent

Col: 21
 E127 continuation line over-indented for visual indent

Tool: Pyflakes
Processed Files:
    setup.py
    rbtools/commands/__init__.py



Tool: PEP8 Style Checker
Processed Files:
    setup.py
    rbtools/commands/__init__.py

This option group would ideally be a global thing. Currently, it is an option group that no command uses.
What happens when you put color_options in _global_options ?

--log-warn-color

LOG_WARNING_COLOR

--log-error-color

LOG_ERROR_COLOR

--log-critical-color

LOG_CRITICAL_COLOR

You also need to check if sys.stderr is a TTY. If it is not (e.g., it is a pipe or a file redirect), you don't want to send the escape sequences.
See https://reviews.reviewboard.org/r/7716/diff/1#0.9 for an example of colorlog being used this way.

You can do:

handler.setFormatter(ColoredFormatter(
    '%(log_color)%s(levelname)-8s(reset)s',
    '...',
    ...))

Single quotes.

Remvoe these.

self.options.warn_color

etc.

Tool: Pyflakes
Processed Files:
    setup.py
    rbtools/commands/__init__.py



Tool: PEP8 Style Checker
Processed Files:
    setup.py
    rbtools/commands/__init__.py

This won't work the way you expect. This alsod oesn't handle the case where stderr is a tty.
You'll want to do:

if (self.options.color == 'always' or
    (self.options.color == 'auto' and
     not sys.stderr.isatty())):

Blank line between these.

This should be an else clause

Cologlog doesnt work on windows without Colorama. We should check the platform and add colorama as a dep in that case.

Tool: Pyflakes
Processed Files:
    setup.py
    rbtools/commands/__init__.py



Tool: PEP8 Style Checker
Processed Files:
    setup.py
    rbtools/commands/__init__.py

I think make 'colorlog' and 'six>=1.8.0' in one line is better than two lines.

Your latest diff doesn't contain any changes.

Tool: Pyflakes
Processed Files:
    setup.py
    rbtools/commands/__init__.py



Tool: PEP8 Style Checker
Processed Files:
    setup.py
    rbtools/commands/__init__.py

I think what you meant to say was something like:
if any(option in self.options.color for option in ('auto', 'always')):

which is a fancier version of:
if 'auto' in self.options.color or 'always' in self.options.color:

Tool: PEP8 Style Checker
Processed Files:
    setup.py
    rbtools/commands/__init__.py



Tool: Pyflakes
Processed Files:
    setup.py
    rbtools/commands/__init__.py

In comparison with the above field set, the fields in 'Option' are lined up with the opening parathesis. Should the fields for OptionGroup also be indented accordingly? or does that make every line exceed the 80 character limit?

We always avoid backslashes if possible. Also, the logic is backwards a bit; if color is always, we always will use colorlog. Otherwise, if it is auto and we are using a TTY, then we will sue it too.
This can be reformatted as:

if (self.options.color == 'always' or
    (self.options.color == 'auto' and
     sys.stderr.isatty())):

Let's reformat this to:

install_requires = [
    'colorama',
    'colorlog',
    'six>=1.8.0',
]

Tool: Pyflakes
Processed Files:
    setup.py
    rbtools/commands/__init__.py



Tool: PEP8 Style Checker
Processed Files:
    setup.py
    rbtools/commands/__init__.py

This should be an else clause otherwise this won't set up the formatter when the options are color == auto and stderr is not a tty.

Put the ] on the next line.
Should end with a comma.

Tool: Pyflakes
Processed Files:
    setup.py
    rbtools/commands/__init__.py



Tool: PEP8 Style Checker
Processed Files:
    setup.py
    rbtools/commands/__init__.py

Minor nitpicks. Looks good to me!

No blank line here.

Please format as:

install_requires = [
    'colorama',
    'colorlog',
    'six>=1.8.0',
]

Tool: Pyflakes
Processed Files:
    setup.py
    rbtools/commands/__init__.py



Tool: PEP8 Style Checker
Processed Files:
    setup.py
    rbtools/commands/__init__.py

Tool: Pyflakes
Processed Files:
    setup.py
    rbtools/commands/__init__.py



Tool: PEP8 Style Checker
Processed Files:
    setup.py
    rbtools/commands/__init__.py