search

avevlad / gulp-connect

Gulp plugin to run a webserver (with live reload)
https://npmjs.org/package/gulp-connect
MIT License
632 stars 113 forks source link

Doesn't follow gulp guidelines. #3

Closed phated closed 10 years ago

phated commented 10 years ago

According to the gulp guidelines, a plugin should do 1 thing.

This plugin does multiple things.

Also, this plugin isn't using the gulp stream in a meaningful way. The better way would to just use this code directly. This should be made a recipe and unpublished.

/cc @Contra @sindresorhus

phated commented 10 years ago

I noticed this is tagged as gulpfriendly instead of gulpplugin, which is good. I think it should drop the gulp- prefix since it is useful in things other than gulp. It is a nice wrapper around connect + open.