Code review comment for lp:~wallyworld/launchpad/recipe-find-related-branches

Revision history for this message
Steve Kowalik (stevenk) wrote :

This looks okay to me, aside from a few blank lines that look to have crept -- however, if make lint disagrees with me, then I'll let that slide.

Why are the imports for lp.testing.factory right under the docstring rather than being ordered with the rest of them?

I'd suggest running utilites/format-imports over the files you changed.

review: Approve (code*)

« Back to merge proposal