Code review comment for lp:~camptocamp/openerp-mrp-reports/7.0-add_serpentcs_mrp_repair_webkit-afe

Lorenzo Battistini (elbati) wrote :

Hello Alexandre,

just checked with https://code.launchpad.net/~savoirfairelinux-openerp/lp-community-utils/checkout-pep8

$ ./checkout-flake8.sh lp:~camptocamp/openerp-mrp-reports/7.0-add_serpentcs_mrp_repair_webkit-afe mrp_repair_webkit
Making a temp directory
branching to /tmp/checkout-flake/~camptocamp.openerp-mrp-reports.7.0-add_serpentcs_mrp_repair_webkit-afe
Branched 3 revisions.

mrp_repair_webkit/__init__.py:24:1: W391 blank line at end of file
mrp_repair_webkit/report/__init__.py:25:1: W391 blank line at end of file
mrp_repair_webkit/report/order.py:25:1: E302 expected 2 blank lines, found 1
mrp_repair_webkit/report/order.py:34:23: E231 missing whitespace after ','
mrp_repair_webkit/report/order.py:37:12: E111 indentation is not a multiple of four
mrp_repair_webkit/report/order.py:39:12: E111 indentation is not a multiple of four
mrp_repair_webkit/report/order.py:49:1: W391 blank line at end of file

mrp_repair_webkit/__init__.py:5:80: E501 line too long (95 > 79 characters)
mrp_repair_webkit/__openerp__.py:5:80: E501 line too long (95 > 79 characters)
mrp_repair_webkit/report/__init__.py:5:80: E501 line too long (95 > 79 characters)
mrp_repair_webkit/report/order.py:5:80: E501 line too long (95 > 79 characters)

Thanks

review: Needs Fixing

« Back to merge proposal