Code review comment for lp:~savoirfairelinux-openerp/knowledge-addons/cmis_write

Revision history for this message
Sandy Carter (http://www.savoirfairelinux.com) (sandy-carter) wrote :

This MP should have a prerequisite branch: lp:~savoirfairelinux-openerp/knowledge-addons/cmis
l.301 missing context propagation

No unittests

Flake8
cmis_write/ir_attachment.py:73:45: W291 trailing whitespace
cmis_write/ir_attachment.py:74:53: W291 trailing whitespace
cmis_write/ir_attachment.py:75:43: W291 trailing whitespace
cmis_write/ir_attachment.py:76:41: W291 trailing whitespace
cmis_write/ir_attachment.py:77:51: W291 trailing whitespace
cmis_write/ir_attachment.py:89:5: E265 block comment should start with '# '
cmis_write/metadata.py:23:1: F401 'osv' imported but unused

review: Needs Fixing

« Back to merge proposal