openerp-community-reviewer team mailing list archive

["Sandy Carter \(http://www.savoirfairelinux.com\)" <sandy.carter@xxxxxxxxxxxxxxxxxxxx>]

Review: Needs Fixing code review, no test

Still some PEP8 issues:
account_invoice_reorder_lines/__init__.py:19:75: W291 trailing whitespace
account_invoice_reorder_lines/__init__.py:23:1: W391 blank line at end of file
account_invoice_reorder_lines/__openerp__.py:21:1: O603 Manifest "license" key is missing
account_invoice_reorder_lines/__openerp__.py:22:11: E203 whitespace before ':'
account_invoice_reorder_lines/__openerp__.py:23:14: E203 whitespace before ':'
account_invoice_reorder_lines/__openerp__.py:24:15: E203 whitespace before ':'
account_invoice_reorder_lines/__openerp__.py:28:13: E203 whitespace before ':'
account_invoice_reorder_lines/__openerp__.py:29:14: E203 whitespace before ':'
account_invoice_reorder_lines/__openerp__.py:30:14: E203 whitespace before ':'
account_invoice_reorder_lines/__openerp__.py:31:11: E203 whitespace before ':'
account_invoice_reorder_lines/__openerp__.py:32:11: E203 whitespace before ':'
account_invoice_reorder_lines/__openerp__.py:33:11: E203 whitespace before ':'
account_invoice_reorder_lines/__openerp__.py:35:19: E203 whitespace before ':'
account_invoice_reorder_lines/__openerp__.py:36:18: E203 whitespace before ':'
-- 
https://code.launchpad.net/~camptocamp/sale-wkfl/7.0-add_reorderable_lines-afe/+merge/197175
Your team Sale Core Editors is requested to review the proposed merge of lp:~camptocamp/sale-wkfl/7.0-add_reorderable_lines-afe into lp:sale-wkfl.