-
Notifications
You must be signed in to change notification settings - Fork 180
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
Update chokidar - macOS fixes #195
Comments
@goyney AFAIR, main reason behind that was chokidar dropping compatibility with Node 6 in the major release. Considering that |
Double-checked - yup, 3.0.0 is Node 8+-only |
I mean, that is what semver is for. Those instances wouldn't update? Seems like this could be a major version bump. EDIT: Also, consider https://nodejs.org/en/about/releases/. Node 6/8 aren't LTS anymore. |
@goyney That is true. However, that would mean that those older instances will stop receiving any fixes at all, unless someone backports them, and doing any changes in forever is painful already. Check these statistics: Node 6 is still very much a thing (even though personally I would very much preferred if it wasn't) |
FYI here's an NPM warning on clean install:
Here's how babel plans to deal with it without dropping node 6 support: babel/babel#11560 |
@Hypnosphi OK, this is pretty smart. Would you be open to submitting a PR implementing the babel solution? |
Please consider updating chokidar to the latest version (this library is a full major version behind). They have just fixed several issues around macOS and Docker that would improve file watching.
The text was updated successfully, but these errors were encountered: