-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
chore(deps): upgrade chokidar #2539
Conversation
Codecov Report
@@ Coverage Diff @@
## v4 #2539 +/- ##
=======================================
Coverage 93.77% 93.77%
=======================================
Files 34 34
Lines 1333 1334 +1
Branches 381 381
=======================================
+ Hits 1250 1251 +1
Misses 81 81
Partials 2 2
Continue to review full report at Codecov.
|
/cc @hiroppy Are you here? |
Can we get a new release out? |
+1, I'm effectively blocked on one of my projects until I lose the dependency on Chokidar 2. |
@paulmillr next week, we are very quickly working on fixes for new next release |
* chore(deps): upgrade chokidar * chore(deps): switch to promise method without async for close * chore(ci): remove node v6 * chore(deps): fix issue of closing watchers before middleware * chore(deps): upgrade chokidar to v3.4.0
@evilebottnawi how is the new release doing? |
For Bugs and Features; did you add new tests?
Not yet
Motivation / Use-Case
upgrade chokidar to v3.3
Breaking Changes
chokidar doesn't support node v6
close should work the same
Additional Info
Edit: didn't see #2533 when I did this, I just applied the patch suggested there