Code review comment for lp:~bladernr/checkbox/1100594-fix-camera_test-new-gir1.2

Revision history for this message
Brendan Donegan (brendan-donegan) wrote :

There seem to be some pep8 errors in your code related to spacing between ',' mainly. I can't remember for certain but I think this may cause Tarmac to reject the submission. Also I'm curious about the comment about fswebcam above - I have fswebcam installed on my system and still get this error. Is there a further problem in the script related to that? Probably best to check it out now.

review: Needs Fixing

« Back to merge proposal