Code review comment for lp:~vila/uci-engine/makedirs-races

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

Looks fine, the only thing I would say is that since this appears to be such a common problem, we should probably just make this a helper function in ci_utils and call it from there. safe_make_dirs() or something.

review: Approve

« Back to merge proposal