Code review comment for lp:~fazerlicourice/music-app/test-empty-library

Revision history for this message
Andrew Hayzen (ahayzen) wrote :

Looking good so far, a few inline comments, I haven't run the code yet these are just observations :-)

Also the code should pass PEP8 and pyflakes, see the link [0] for the current errors.

And could you add an entry for yourself in the debian/changelog (simply run $ dch) and link bug 1261587 as has been done in other entries.

0 - http://pastebin.ubuntu.com/14082325/

review: Needs Fixing

« Back to merge proposal