Code review comment for lp:~lasall/ppa-purge/dryrun
Revision history for this message
| Tormod Volden (tormodvolden) wrote | # |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
67d34a1
(Get the code!)
Hi Dominique,
Can you rebase 59-60-61 into one new commit? It is just a bit confusing with commits undoing another.
You have also spelled "disable" wrong at least twice.
Are you serious about checking for 9223372036854775807 backup files? If loads of backup files are already present it will be wrong to add another IMO.
If you are not reusing NUM it is not necessary to unset it.