holmgr / cargo-sweep

A cargo subcommand for cleaning up unused build files generated by Cargo
MIT License
693 stars 31 forks source link

`cargo sweep -d --file` removes timestamp file #40

Closed Nemo157 closed 1 year ago

Nemo157 commented 3 years ago

I expected it to be possible to do a dry-run, then an actual run afterwards, but the dry-run deleted the timestamp file so that the actual run didn't work.

holmgr commented 3 years ago

Sounds like something we want to fix

marcospb19 commented 1 year ago

Note for someone who might solve this in the future, you might want to update this test too: https://github.com/holmgr/cargo-sweep/blob/fd4dc55dd89d480276aace44b1af6adeeab7ed05/tests/integration.rs#L192-L194