Code review comment for lp://staging/~joetalbott/qa-dashboard/update_script_chdir

Revision history for this message
Andy Doan (doanac) wrote :

this is fine, but a nice trick that has less side-effects in your code would be to use the "cwd" parameter:

 if subprocess.call(['bzr', 'missing'], cwd=script_dir, stdout=FNULL):

« Back to merge proposal