Code review comment for lp:~jcsackett/launchpad/anonymous-api-access-emails-681815

Revision history for this message
Edwin Grubbs (edwin-grubbs) wrote :

Hi JC,

Thanks for adding tests that actually fail without your changes to checkUnauthenticated(). I like benji's suggestion to wrap lines in the doctests to make lint complain less.

-Edwin

review: Approve (code)

« Back to merge proposal