Code review comment for lp:~bigkevmcd/landscape-client/517453-process-race-condition

Revision history for this message
Thomas Herve (therve) wrote :

Cool! Just one last thing:

[3] As you regrouped with only one finally, you need to close the file after each operation.

+1!

review: Approve

« Back to merge proposal