-
Notifications
You must be signed in to change notification settings - Fork 35
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
add dry run option #230
Labels
Comments
I added a patch to test this for purging, and plan to implement it for just regular rmw'ing. |
The -n, --dry-run options haven't been added yet but that'll happen before the next release. |
andy5995
added a commit
that referenced
this issue
Aug 8, 2019
andy5995
added a commit
that referenced
this issue
Aug 27, 2020
zvezdochiot
pushed a commit
to FS-make-simple/rmw
that referenced
this issue
Sep 5, 2022
enabled with RMWTRASH=dry-run
zvezdochiot
pushed a commit
to FS-make-simple/rmw
that referenced
this issue
Sep 5, 2022
zvezdochiot
pushed a commit
to FS-make-simple/rmw
that referenced
this issue
Sep 5, 2022
…astronaut#230) This is still very experimental!
zvezdochiot
pushed a commit
to FS-make-simple/rmw
that referenced
this issue
Sep 5, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Adding a dry run (-n) option for purge was discussed on IRC today. I think ultimately that would be a good feature. I don't feel a need for it personally because for my workflow, if I have stuff in the trash for 45 or 90 days and haven't missed it, I don't worry that I need to review what will be purged.
But I could see how some users might like that feature and would consider adding it in the future or accepting a PR to add it.
The text was updated successfully, but these errors were encountered: