Skip to content
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

Feature request: ability to delete files with massren #11

Closed
315234 opened this issue Feb 19, 2014 · 5 comments
Closed

Feature request: ability to delete files with massren #11

315234 opened this issue Feb 19, 2014 · 5 comments

Comments

@315234
Copy link

315234 commented Feb 19, 2014

For example, file names which are replaced with blank lines or some kind of flag ("DELETE_ME" etc) could be deleted rather than renamed. This would be useful for cleaning up messy directories, for example after extracting an archive in the wrong place. Could have a command line flag to turn this feature on.

@leifg
Copy link

leifg commented Feb 20, 2014

👍 I thought of the exact same thing the first time I tested the tool.

But nevertheless, great work.

@laurent22
Copy link
Owner

Yes I think that would be a useful feature and probably not hard to implement. I'm not too keen on deleting files replaced by blank lines as it could lead to accidental deletion. I'd rather make it explicit. For example, any line replaced by a special comment could be deleted.

For example, given these files:

file1
file2
file3

If it's changed to this:

file1
//rm
file3

It means "file2" would be deleted.

One caveat though is that the undo function would not work for deletion (unless a lot of system-specific stuff is added in order to move the files to the trash) so that would probably have to be documented.

@laurent22
Copy link
Owner

This is now available in v1.2.0 - if you add "//" before a filename it will be deleted. Please give it a try if you can, and let me know if you notice any problem.

@laurent22
Copy link
Owner

Implemented in 1.2.0

@laurent22 laurent22 reopened this Mar 22, 2014
@tborychowski
Copy link

this should be documented. I've spent a while looking for this.
The main readme mentioning that it is possible, it doesn't say how...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants