Code review comment for lp://staging/~abentley/bzr/branches-uncommitted

Revision history for this message
John A Meinel (jameinel) wrote :

69 + prefix += 'U' if uncommitted else ' '

Given the line just before this is:

if active:
  prefix =
else:
  prefix =

I would probably follow suite here. (or change the other one to be consistent).

Otherwise seems good.

review: Approve

« Back to merge proposal