Code review comment for lp:~salgado/launchpad/remove-security-upload-policy

Revision history for this message
Jeroen T. Vermeulen (jtv) wrote :

Wow, that's a very red diff. Death to doctests!

Not much here to review, really. I'll have to trust that you're not bluffing on how an email to Daniel Silverstone turned into one to Celso Providelo in lines 286—307 of the diff. But then again, doctests have a tendency to invite bluffing and I can't help feeling it's their own fault.

review: Approve

« Back to merge proposal