openerp-community-reviewer team mailing list archive
-
openerp-community-reviewer team
-
Mailing list archive
-
Message #01905
Re: lp:~camptocamp/stock-logistic-flows/7.0-add_stock_picking_priority-afe into lp:stock-logistic-flows/7.0
Review: Needs Fixing code review
__openerp__: pep8 issues
* spaces before the colon on l.56,57,58,59,66,67,68,71
picking_priority.py:
* l.123 you're not checking if ids is an int or a list, this could lead to a problem with search and an error on l.137
--
https://code.launchpad.net/~camptocamp/stock-logistic-flows/7.0-add_stock_picking_priority-afe/+merge/197206
Your team Stock and Logistic Core Editors is requested to review the proposed merge of lp:~camptocamp/stock-logistic-flows/7.0-add_stock_picking_priority-afe into lp:stock-logistic-flows/7.0.
References