2 ms·
> To list backups that will should be kept use the --invert option. May I suggest changing the name to something more direct? Calling it "--invert" means the u
by adrianmonk 7y ago
> To list backups that will should be kept use the --invert option.
May I suggest changing the name to something more direct? Calling it "--invert" means the user must think through what the default sense of the test is, then negate that in their mind. Not exactly a tough mental task, but people do make careless errors.
Perhaps something like "--list=keep" and "--list=discard" (with the default being "discard").
Also, typos:
"will make prunef to keep" --> "will make prunef keep"
"list backups that will should be" --> "list backups that should be"
- closeneough 7y agoThanks for the input and for pointing out the typos. They are fixed now. I will consider changing the invert flag, but I'm not that happy with something like "--list=...". There will be only two modes with discard being the default one. So imho there should be only one flag to switch to the non-default mode.
- closeneough 7y agoI've decided for --list-kept. --invert will be supported to not break scripts.