Code review comment for lp://qastaging/~manishsinha/software-properties/add-update-after-repo-add

Revision history for this message
Michael Terry (mterry) wrote :

44 + print("YAY")

Should be dropped.

52 - sp.sourceslist.save()
53 + if options.assume_update:
54 + sp.sourceslist.save()

Why does the save move inside the if? It seems like without the -u, you should still save.

review: Needs Fixing

« Back to merge proposal