← Back to team overview

openerp-community-reviewer team mailing list archive

Re: [Merge] lp:~camptocamp/account-invoicing/account-analytic-required-vre into lp:account-invoicing

 

Review: Needs Fixing

Thanks Vincent,

some style fixes:

account_invoice_analytic_required/__init__.py:3:2: W291 trailing whitespace
account_invoice_analytic_required/__init__.py:19:75: W291 trailing whitespace
account_invoice_analytic_required/__init__.py:23:14: E271 multiple spaces after keyword
account_invoice_analytic_required/__openerp__.py:3:2: W291 trailing whitespace
account_invoice_analytic_required/__openerp__.py:19:75: W291 trailing whitespace
account_invoice_analytic_required/__openerp__.py:23:1: O603 Manifest "license" key is missing
account_invoice_analytic_required/__openerp__.py:24:11: E203 whitespace before ':'
account_invoice_analytic_required/__openerp__.py:25:14: E203 whitespace before ':'
account_invoice_analytic_required/__openerp__.py:26:15: E203 whitespace before ':'
account_invoice_analytic_required/__openerp__.py:28:13: E203 whitespace before ':'
account_invoice_analytic_required/__openerp__.py:29:14: E203 whitespace before ':'
account_invoice_analytic_required/__openerp__.py:30:14: E203 whitespace before ':'
account_invoice_analytic_required/__openerp__.py:32:11: E203 whitespace before ':'
account_invoice_analytic_required/__openerp__.py:33:11: E203 whitespace before ':'
account_invoice_analytic_required/__openerp__.py:34:11: E203 whitespace before ':'
account_invoice_analytic_required/__openerp__.py:35:18: E203 whitespace before ':'
account_invoice_analytic_required/__openerp__.py:36:19: E203 whitespace before ':'
account_invoice_analytic_required/__openerp__.py:37:18: E203 whitespace before ':'
account_invoice_analytic_required/account_invoice_analytic_required.py:23:1: O603 Manifest "license" key is missing
account_invoice_analytic_required/account_invoice_analytic_required.py:25:1: E302 expected 2 blank lines, found 1
account_invoice_analytic_required/account_invoice_analytic_required.py:27:13: E225 missing whitespace around operator

account_invoice_analytic_required/__openerp__.py:24:80: E501 line too long (84 > 79 characters)
account_invoice_analytic_required/__openerp__.py:27:80: E501 line too long (94 > 79 characters)
account_invoice_analytic_required/account_invoice_analytic_required.py:30:80: E501 line too long (110 > 79 characters)
-- 
https://code.launchpad.net/~camptocamp/account-invoicing/account-analytic-required-vre/+merge/218974
Your team Account Core Editors is subscribed to branch lp:account-invoicing.


References