Review: Needs Fixing

Wait, I'm not suggesting preparing 6 MPs right now. I suggesting only 
implementing --dry-run for now, then we will see what to do next.

Basically we want an argument like:

  add_argument('--dry-run', action='store_true')

and then make

  copy_result(path, args.old_release, args.new_release)

to:

  if not args.dry_run:
      copy_result(path, args.old_release, args.new_release)

Check for other "write operations" to skip, but that should basically be it. 
The diff should be of just a few lines.

-- 
https://code.launchpad.net/~andersson123/autopkgtest-cloud/+git/autopkgtest-cloud/+merge/443597
Your team Canonical's Ubuntu QA is subscribed to branch 
autopkgtest-cloud:master.


-- 
Mailing list: https://launchpad.net/~canonical-ubuntu-qa
Post to     : canonical-ubuntu-qa@lists.launchpad.net
Unsubscribe : https://launchpad.net/~canonical-ubuntu-qa
More help   : https://help.launchpad.net/ListHelp

Reply via email to