Skip to content

chore(deps): upgrade chokidar to v4#407

Closed
stianjensen wants to merge 1 commit intofi3ework:mainfrom
stianjensen:chokidar-4
Closed

chore(deps): upgrade chokidar to v4#407
stianjensen wants to merge 1 commit intofi3ework:mainfrom
stianjensen:chokidar-4

Conversation

@stianjensen
Copy link
Contributor

https://github.com/paulmillr/chokidar/releases/tag/4.0.0

The only breaking changes are that it removes support for globs (not used here), and ups minimum required node version to 14.x (same as already used here).

@netlify
Copy link

netlify bot commented Oct 6, 2024

Deploy Preview for vite-plugin-checker ready!

Name Link
🔨 Latest commit 8e56af2
🔍 Latest deploy log https://app.netlify.com/sites/vite-plugin-checker/deploys/67b37832ff62c9000849cd7f
😎 Deploy Preview https://deploy-preview-407--vite-plugin-checker.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@stianjensen
Copy link
Contributor Author

Unsure if/how the snapshot changes here are related to the Chokidar upgrade 🤔

@felixranesberger
Copy link

Any updates @fi3ework ?

@danielroe danielroe changed the title Upgrade Chokidar to 4.x chore(deps): upgrade chokidar to v4 Feb 16, 2025
@danielroe
Copy link
Collaborator

danielroe commented Feb 16, 2025

The snapshot changes are indicate that the watcher isn't picking up on some changes - likely we need to update more than just the version of chokidar here. (Globs are used internally.) Do you have time to take a look?

@stianjensen
Copy link
Contributor Author

Ah, right – I overlooked the fact that globs are used in some of the watcher.add calls after initially creating the watcher!
I can come back to this as soon as a good general chokidar-glob solution is available, then.

https://github.com/paulmillr/chokidar/releases/tag/4.0.0

The most important breaking change is that it removes support for globs,
which'll need a new solution on top. Then it updates minimum required
node version to 14.x (same as already used here).
@pkg-pr-new
Copy link

pkg-pr-new bot commented Feb 17, 2025

Open in Stackblitz

npm i https://pkg.pr.new/vite-plugin-checker@407

commit: 8e56af2

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.

3 participants