openerp-community-reviewer team mailing list archive

[Lorenzo Battistini - Agile BG <lorenzo.battistini@xxxxxxxxxxx>]

Review: Needs Fixing

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
-- 
https://code.launchpad.net/~camptocamp/openerp-mrp-reports/7.0-add_serpentcs_mrp_repair_webkit-afe/+merge/209044
Your team OpenERP Community Reviewer/Maintainer is requested to review the proposed merge of lp:~camptocamp/openerp-mrp-reports/7.0-add_serpentcs_mrp_repair_webkit-afe into lp:openerp-mrp-reports.