Code review comment for lp:~mikefletcher/mago/gnome-screenshot

Revision history for this message
James Tatum (jtatum) wrote :

It looks good overall. The meat of the LDTP commands in mago/application/gnome.py are all wrapped in try/catch blocks to provide a helpful failure description for the log file. +1 to the comments for moving as many parameters as possible to XML.

review: Needs Fixing

« Back to merge proposal