dialer-app

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

  1. custom_proxy_objects
  2. Merge into trunk
Revision history for this message
Leo Arias (elopio) 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.

Done. Thanks for the review.

« Back to merge proposal