kellin commented on the pull-request: `add script for pruning atomic ostree repos` that
you are following:
``
What I don't like is the manual setting of args.test = True in the body.
Ideally, what I'd *prefer* here is that the script when invoked always goes dry-run
and we have some option that actually performs the action.
I just can't remember the right word for the standard on this...
EG: --dry-run for scripts whose behavior defaults to run in production, and something else
for not.
The idea of having something we have to delete from code bothers me; we should make sure
we set it --dry-run in the cron job or, since this is a destructive script, opt to make
the default behavior not do anything but show the changes and then a single option
--really-do-it (but that's a terrible name) that does the changes for real.
``
To reply, visit the link below or just reply to this email
https://pagure.io/releng/pull-request/7366