Skip to content

Added --watch-poll #4147

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

Closed
wants to merge 1 commit into from
Closed

Added --watch-poll #4147

wants to merge 1 commit into from

Conversation

ilyahoilik
Copy link

There is problem with npm run watch (webpack doesn't detects a changes) without --watch-poll. I use Homestead.

There is problem with `npm run watch` (webpack doesn't detects a changes) without `--watch-poll`. I use Homestead.
@taylorotwell
Copy link
Member

@JeffreyWay thoughts?

@JeffreyWay
Copy link
Contributor

File watching is always a little strange with VMs. Some people have no issues, others need to opt for polling. Polling, as I understand it, puts a bit more strain on things, so it probably shouldn't be the default.

Maybe we can make it more clear in the docs that --watch-poll is an alternative. I'll include that with my Mix PR.

@mlantz
Copy link

mlantz commented Feb 18, 2017

Could you not add another script called watch-poll which would leave it built in but as an addition rather than have people have to customize

@taylorotwell
Copy link
Member

That might be a better option.

@ilyahoilik ilyahoilik deleted the patch-1 branch February 21, 2017 16:54
JeffreyWay added a commit that referenced this pull request Feb 21, 2017
Per #4147, this adds a new `npm run watch-poll` command, and dries up some of those long repetitive commands.
@JeffreyWay JeffreyWay mentioned this pull request Feb 21, 2017
bkrukowski pushed a commit to bkrukowski/laravel that referenced this pull request Mar 10, 2017
Per laravel#4147, this adds a new `npm run watch-poll` command, and dries up some of those long repetitive commands.
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

Successfully merging this pull request may close these issues.

4 participants