Code review comment for lp:~abentley/launchpad/find-merge-directive

Revision history for this message
Gavin Panella (allenap) wrote :

Looks good. Two questions:

 * The test docstrings look like they've been copy-n-pasted.

 * Is it necessary to commit at the end of each test? If so, could you add a short comment explaining why?

review: Approve (code)

« Back to merge proposal