When using scripts/sync-results.py with --dryrun, I found that the printed messages could be misleading. In particular, I don't think we want to tell folks files have been downloaded or are now available on S3 if they use --dryrun. I could be persuaded that "Sync complete" should be outside of this logic.
Purpose/implementation Section
When using
scripts/sync-results.py
with--dryrun
, I found that the printed messages could be misleading. In particular, I don't think we want to tell folks files have been downloaded or are now available on S3 if they use--dryrun
. I could be persuaded that "Sync complete" should be outside of this logic.