Code review comment for lp:~canonical-platform-qa/dialer-app/custom_proxy_objects

Revision history for this message
Chris Gagnon (chris.gagnon) wrote :

Looks good but can you fix the pep8 issues? (You didn't introduce them)

ListItemWithActions.py:37:1: E302 expected 2 blank lines, found 1
tests/test_calls.py:72:80: E501 line too long (86 > 79 characters)

If there were a lot I would ask for it to be fixed in a different MP, But it's only 2.

review: Needs Fixing

« Back to merge proposal