Tool: Pyflakes
Processed Files:
    rbtools/commands/status.py



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

For the arguments that get passed to the max method, I think that it would be more Pythonic if we replaced the list comprehensions with generator expressions (that is, we can simply drop the outermost square brackets). That way, we aren't allocating memory for an intermediate list; instead, each integer is being generated on the fly. You might find this article useful: Code Like a Pythonista: Idiomatic Python

Tool: Pyflakes
Processed Files:
    rbtools/commands/status.py



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

I like the output, but it may be too much if you have a lot of review requests. Currently, they only take up one line each so its easy to get a summary at a glance. I think it would be better if it were formatted like:

=============================================
Status           Review Request
=============================================
Draft            r/1234 - Some Review Request
Ship it!         r/1235 - Some Review Request
Open Issues (4)  r/1236 - Some Review Request

Needs a docstring. Docstrings should be of the form

"""Single line summary.

Multi-line description.
"""

Since you are passing a generator, you only need one set of parenthesis.

Style nitpick: blank line between these.

Tool: Pyflakes
Processed Files:
    rbtools/commands/status.py



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

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

Col: 33
 E703 statement ends with a semicolon

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



Tool: Pyflakes
Processed Files:
    rbtools/commands/status.py

This should be written in the imperative mood (something like "Print review request statuses in a table").

Python str has a nifty ljust method, which can simplify this quite a bit:

white_space = max_status_length + TAB_SIZE

for row in data:
    print '%s%s' % (row[0].ljust(white_space), row[1])


You can use it above for the header, too.

Docstrings should have a summary that fits entirely on one line and then a longer description (if necessary). They should also be written in the imperative mood (So "Get the current status" rather than "Gets ...").

These should use single quotes.

We should only say "Ship It" if there are one or more "Ship It" reviews. By default we should probably just say something like "Pending" or "Open".

It might be better to calculate this in the tabulate method, or to calculate all the lengths here.
It's also not great to stuff a bunch of stuff into a list like this. It means that people reading the tabulate method have to then go read this method to figure out what's in the list. Perhaps use a dictionary instead?

detail = 'r/%s - %s' % (request.id, request.summary)
review_data.append({
    'status': status,
    'detail': detail,
    'status_len': len(status),
    'detail_len': len(detail),
    'row_len': len(status) + len(detail),
})

Tool: Pyflakes
Processed Files:
    rbtools/commands/status.py



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

These could be formatted more nicely by being a bit less verbose and wrapping differently:

max_status_length = max(d['status_len'] for d in data)
max_row_length = (max(d['summary_len'] for d in data) +
                  PADDING + TAB_SIZE + max_status_length)

Instead of using end='', let's just print both of these at the same time:

print('%s%s' % ('Status'.ljust(white_space), 'Review Request'))

Tool: Pyflakes
Processed Files:
    rbtools/commands/status.py



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

Can you update the example output in the description?
Also, in your summary, the plural of "status" is "statuses" (possesive would use status')

Can you document the arguments to this function?
We document our args as follows:

"""...

Args:
    arg_name (arg_type):
        Description of arg.
"""


You don't need to document self.

Can you comment on what these do?

Can you format this like:

            status = request['status'].ljust(white_space)
            print('%s%s' % (status, request['summary'])

You don't need the quotes here. print() should work.

Can you document the args and return of the function as follows:
```

"""...
Args:

    arg_name (arg_type):

        Description of Arg
Returns:

    return_type: Description of return value

"""

"Return" and not "Get". "Get" implies that we are querying for informatino.

Blank line between these.
Also, data is really generic.

data is very generic. Can we call this something else? Perhaps request_stats?

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



Tool: Pyflakes
Processed Files:
    rbtools/commands/status.py

Can you put the comment directly before each one, e.g.

# The number of spaces between a request status ...
TAB_SIZE = 3

# THe number of ...
PADDING = 5


These should also probably be class constants, e.g.

class Status(Command):
    # ...

    # The number of spaces...
    TAB_SIZE = 3


and you can refer to them with either Status.TAB_SIZE or self.TAB_SIZE.

Tool: Pyflakes
Processed Files:
    rbtools/commands/status.py



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