Code review comment for lp:~wallyworld/launchpad/recipe-find-related-branches
Revision history for this message
| Ian Booth (wallyworld) wrote | # |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
0e1f616
(Get the code!)
lint errors (almost all line length) were there already and since this
code base is relatively new, I figured I'd leave well enough alone.
I'll see if I can tweak the zcml instead of using removeSecurityProxy
Thanks.
On 03/12/10 15:01, Steve Kowalik wrote:
> Review: Approve code*
> This looks great! My only concerns are the lint errors, and the (it seems to me) excessive use of removeSecurityP