Update chokidar to ^3.2.1#371
Update chokidar to ^3.2.1#371ssk101 wants to merge 1 commit intobrowserify:masterfrom ssk101:chokidar-3.2.1
Conversation
|
@goto-bus-stop is there anything I can help with to move this forward? |
|
The travis-ci will never pass, because the project tests 5 old versions of nodejs, but fsevents 2.x used by chokidar 3.x only supports nodejs 8+: |
|
We have had to skip testing watchify on MacOS in the node.js smoke test suite CITGM due to failures related to this old dependency |
|
Now first security warnings arrived for dependencies of the old chockidar 2.1.x version. Now this update may get a bit more priority i hope... Watchify depends on watchify@3.11.1 > chokidar@2.1.8 > braces@2.3.2 > snapdragon@0.8.2 > base@0.11.2 > cache-base@1.0.1 And - another positive side effect: using chockidar 3.x instead of 2.1 dramatically reduces dependency count (see https://paulmillr.com/posts/chokidar-3-save-32tb-of-traffic/) Security warning: https://snyk.io/vuln/SNYK-JS-CACHEBASE-1054631 |
|
Addon - |
|
merged in #378 and released as 4.x. Thanks for the PR! |
Older versions of Chokidar have a few issues watching large amounts of files (as in no events are triggered on file changes after a while).