Code review comment for lp:~lifeless/bzr/repo-source

Martin Pool (mbp) wrote :

I'd like the test to have the URL of the bug it relates to, as a pointer in case it regresses or people want more information about the test.

It seems plausible to me but it's not obvious why _get_keys should omit ghosts. Maybe the docstring for get_keys or PendingAncestryResult should say so?

review: Needs Fixing

« Back to merge proposal