Code review comment for lp://staging/~camptocamp/department-mgmt/add-account-department-fix-default-anyltic-jge
Revision history for this message
| Pedro Manuel Baeza (pedro.baeza) wrote | # |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
2eb648a
demo site
(Get the code!)
FYI, first form doesn't comply with PEP8 indentation rule that if you put arguments of first line, you have to put vertical alignment, but the second one lacks some spaces after commas ;)
That it's marked as E128 error on flake8.
Regards.