Hacker News new | ask | show | jobs
by adrianmonk 2307 days 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"

1 comments

Thanks 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.

I've decided for --list-kept. --invert will be supported to not break scripts.