Code review comment for lp://staging/~parthm/bzr/571467-ci-show-diff-short-opt-p

Revision history for this message
Andrew Bennetts (spiv) wrote :

I had to read the bug to figure out why "-p" (it's short for "patch"). It'd be nice to explain that somewhere, but I don't know where would be appropriate. Anyway, this seems fine.

review: Approve

« Back to merge proposal