Code review comment for lp:~cprov/adt-cloud-worker/uci-nova-refix

Revision history for this message
Paul Larson (pwlars) wrote :

I'm actually still not 100% sure of the logic there, and it looks like you did something different from what Evan recommended but Francis said he was able to test it and see it work too. So, I'd like to see it land if it works, but maybe just add a quick comment about what it's doing there since there seems to be some confusion about it.

review: Approve

« Back to merge proposal