Va1 / browser-sync-webpack-plugin

Easily use BrowserSync in your Webpack project.
MIT License
369 stars 40 forks source link

Move browser-sync to peerDependencies #2

Closed bjrmatos closed 9 years ago

bjrmatos commented 9 years ago

I think is a good idea to move browser-sync to peerDependencies to keep track of automatic updates in the module

Va1 commented 9 years ago

OK, makes sense. Thank you!

Va1 commented 9 years ago

Done with https://github.com/Va1/browser-sync-webpack-plugin/commit/898abc2659342a2d80f28ffef89e7df51f3f00ad . Closing.

bjrmatos commented 9 years ago

:clap:

bjrmatos commented 9 years ago

i think that >=2.x.x <3 or ^2.x.x will be much better

Va1 commented 9 years ago

The reason I've put there latest versions is to be sure it will work, as I used the API of latest versions during development of the package.

If you did test it with 2.0.0 and sure that it works, I will change this (or you can submit a PR).

bjrmatos commented 9 years ago

cool, let me test in browser-sync 2.0.0, if it is still working i'll send a PR