redux-filter icon indicating copy to clipboard operation
redux-filter copied to clipboard

T-shirt example uses BrowserSync

Open barrystaes opened this issue 9 years ago • 1 comments

I'd like to suggest to remove BrowserSync from the examples.

As a new user looking for a quick example, i do not see the value it adds here. But i do experience a lot of overhead;

  • on Windows it requires Visual Studio and Python installed.
  • on Linux it starts arguing with the browsersync instance i already run.
  • on Linux it slows everything down, i dont like staring at npm install and npm start forever. (i did a lot of those to pinpoint #25)

barrystaes avatar Feb 16 '16 11:02 barrystaes

Some fair points. I personally like the live reloading, hot reloading. I think it encourages experimentation.

However, I don't want to hinder experimentation by slow installs. I'd be more than willing to accept a PR that removes browser sync. We can replace it with the built in webpack server. I'm also open to other suggestions.

It'll be at least until the weekend until I can undertake a replacement this large.

nsmith7989 avatar Feb 16 '16 17:02 nsmith7989