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

Deprecate #14

Closed
sindresorhus opened this issue Jun 25, 2014 · 11 comments
Closed

Deprecate #14

sindresorhus opened this issue Jun 25, 2014 · 11 comments

Comments

@sindresorhus
Copy link

This plugin should be deprecated as the usage is awkward and it's not really a conforming gulp plugin. There's are lots of node modules that can do the same in a less awkward matter.

@contra @phated

@yocontra
Copy link

I agree, the only case though is if you want to pipe to rimraf as a part of a pipeline, for example you gulp.src('src.tar.gz').pipe(gulp.dest('dest')).pipe(rimraf()) to essentially mv a file

@robrich
Copy link
Owner

robrich commented Jun 25, 2014

If you want to move a file, shell commands seems a lighter weight way to do
it.

What other tools are better suited to deleting some but not all files from
a dir?

@sindresorhus
Copy link
Author

@robrich Eg. del supports multiple globbing patterns.

So:

del(['tmp/*.js', '!tmp/unicorn.js'], cb);

@robrich
Copy link
Owner

robrich commented Jun 25, 2014

Cool -- del combines globbing and rimraf.

I'm good with this. gulp-shell, gulp-script, gulp-run, and gulp-clean are already blacklisted. I think we'll need a recipe to show del in action (though the README script goes a long way towards this). All else being equal, recipes and plugins should be searchable via gulpjs.com/plugins as I've seen many ask questions answered by recipes (e.g. yet another "remove gulp-browserify" thread.)

@sindresorhus
Copy link
Author

I think we'll need a recipe to show del in action

Yes, I've promised @contra to do one.

All else being equal, recipes and plugins should be searchable via gulpjs.com/plugins as I've seen many ask questions answered by recipes (e.g. yet another "remove gulp-browserify" thread.)

I agree, but lets discuss that on the plugins tracker.

shinnn added a commit to shinnn/heading-level.js that referenced this issue Jun 28, 2014
shinnn added a commit to shinnn/node-flex-svg that referenced this issue Jun 28, 2014
mguterl added a commit to mguterl/laurelandmike that referenced this issue Jul 8, 2014
alrra pushed a commit to h5bp/html5-boilerplate that referenced this issue Aug 5, 2014
@sindresorhus
Copy link
Author

ping @robrich

There's now a gulp recipe for using del: https://github.com/gulpjs/gulp/blob/master/docs/recipes/delete-files-folder.md

@sindresorhus
Copy link
Author

ping

@robrich
Copy link
Owner

robrich commented Sep 24, 2014

Ack

@robrich
Copy link
Owner

robrich commented Sep 24, 2014

The recipe looks awesome, this plugin is now also blacklisted. Is there more to do?

@sindresorhus
Copy link
Author

Just change the repo/readme description to "[deprecated] rimraf plugin for gulp" and link to the recipe.

@phated
Copy link

phated commented Sep 24, 2014

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