openerp-community-reviewer team mailing list archive

Thread
Date

Re: [Merge] lp:~camptocamp/department-mgmt/add-account-department-fix-default-anyltic-jge into lp:department-mgmt

To: Joël Grand-Guillaume @ camptocamp <joel.grandguillaume@xxxxxxxxxxxxxx>
From: Pedro Manuel Baeza <pedro.baeza@xxxxxxxxx>
Date: Tue, 13 May 2014 13:05:28 -0000
In-reply-to: <20140508141929.2439.32806.launchpad@ackee.canonical.com>
Reply-to: mp+218807@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

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.
-- 
https://code.launchpad.net/~camptocamp/department-mgmt/add-account-department-fix-default-anyltic-jge/+merge/218807
Your team Department Core Editors is subscribed to branch lp:department-mgmt.

References

[Merge] lp:~camptocamp/department-mgmt/add-account-department-fix-default-anyltic-jge into lp:department-mgmt
From: Joël Grand-Guillaume, 2014-05-08