Code review comment for lp:~elopio/ubuntu-clock-app/refactor_timer_tests

Revision history for this message
Nicholas Skaggs (nskaggs) wrote :

So most of this is actually pep8 and pyflakes. Don't worry Leo, it bothers me as well (even though I don't like pep8 in some cases).

Most of the restructuring is straightforward; really nothing crazy. It gets my +1. I will say, it's difficult to read the diff because of the pep8 and pyflakes changes, but I understand the desire to lump them.

review: Approve

« Back to merge proposal