This is a review from Review Bot.
  Tool: PEP8 Style Checker
  Processed Files:
    rbtools/postreview.py
    rbtools/utils/users.py
  Ignored Files:

Col: 80
 E501 line too long (81 > 79 characters)

Col: 1
 E302 expected 2 blank lines, found 1

Col: 19
 E231 missing whitespace after ':'

Col: 31
 E231 missing whitespace after ':'

Col: 43
 E231 missing whitespace after ':'

Col: 25
 E241 multiple spaces after ','

Col: 37
 E241 multiple spaces after ','

Col: 18
 E231 missing whitespace after ':'

Col: 33
 E231 missing whitespace after ':'

Col: 25
 E241 multiple spaces after ','

Col: 16
 E711 comparison to None should be 'if cond is None:'

Col: 66
 E502 the backslash is redundant between brackets

This is a review from Review Bot.
  Tool: Pyflakes
  Processed Files:
    rbtools/postreview.py
    rbtools/utils/users.py
  Ignored Files:

This is a review from Review Bot.
  Tool: PEP8 Style Checker
  Processed Files:
    rbtools/postreview.py
    rbtools/utils/users.py
  Ignored Files:

Col: 1
 E302 expected 2 blank lines, found 1

This is a review from Review Bot.
  Tool: Pyflakes
  Processed Files:
    rbtools/postreview.py
    rbtools/utils/users.py
  Ignored Files:

This is a review from Review Bot.
  Tool: PEP8 Style Checker
  Processed Files:
    rbtools/postreview.py
    rbtools/utils/users.py
  Ignored Files:

This is a review from Review Bot.
  Tool: Pyflakes
  Processed Files:
    rbtools/postreview.py
    rbtools/utils/users.py
  Ignored Files:

This should probably check sys.__stdin__.isatty() and just automatically make a decision without any output in the case where python is not running in an interactive terminal.

This should probably use distutils.util.strtobool instead of rolling it yourself.

This is a review from Review Bot.
  Tool: PEP8 Style Checker
  Processed Files:
    rbtools/postreview.py
    rbtools/utils/users.py
  Ignored Files:

This is a review from Review Bot.
  Tool: Pyflakes
  Processed Files:
    rbtools/postreview.py
    rbtools/utils/users.py
  Ignored Files:

These should still be broken up into two groups (one group for standard library modules, and one for rbtools). They should also be sorted alphabetically, so the distutils line should come first.

Please capitalize "Invalid"

Please move this block to come first--there's no reason to define the prompt if we never use it.

raw_input will print a prompt, so this can be condensed to:

choice = raw_input(question + prompt).lower()

This is a review from Review Bot.
  Tool: PEP8 Style Checker
  Processed Files:
    rbtools/postreview.py
    rbtools/utils/users.py
  Ignored Files:

This is a review from Review Bot.
  Tool: Pyflakes
  Processed Files:
    rbtools/postreview.py
    rbtools/utils/users.py
  Ignored Files:

This is a review from Review Bot.
  Tool: PEP8 Style Checker
  Processed Files:
    rbtools/postreview.py
    rbtools/utils/users.py
  Ignored Files:

This is a review from Review Bot.
  Tool: Pyflakes
  Processed Files:
    rbtools/postreview.py
    rbtools/utils/users.py
  Ignored Files:

We'll need a similar change to the postreview.py one in rbtools/commands/post.py

Hmm. So make_diff is only defined in the git client, which means I think this will explode for anything not git. Can you test with other SCMs and fix things up?

Sorry....Just remember I have a pending review here 4 months agao..... Happy holiday!