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

T-shirt example uses BrowserSync #26

Open
barrystaes opened this issue Feb 16, 2016 · 1 comment
Open

T-shirt example uses BrowserSync #26

barrystaes opened this issue Feb 16, 2016 · 1 comment

Comments

@barrystaes
Copy link

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 T-shirt example broken since v2.0.0 #25)
@nsmith7989
Copy link
Owner

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.

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

2 participants