Hi Yannick,

The comment has been fixed and a docstring added to multi_statement_import.

> l.44 you can use self.pool[''] instead of self.pool.get('')

self.pool.get is used in a lot of places... Do you want it to be replaced 
everywhere? It's just a copy/paste of l.82

> l.47 + l.53 > 80 characters
Do we need to pay attention to this pep8 rule?

Regards,

lmi
-- 
https://code.launchpad.net/~acsone-openerp/banking-addons/bank-statement-reconcile-70-multi-statements/+merge/219683
Your team Banking Addons Core Editors is subscribed to branch 
lp:banking-addons/bank-statement-reconcile-7.0.

-- 
Mailing list: https://launchpad.net/~banking-addons-team
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~banking-addons-team
More help   : https://help.launchpad.net/ListHelp

Reply via email to