-
-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Add node.js only parcel watcher watchman back-end #9789
Conversation
cc @jdlm-stripe |
b00711b
to
5099a0d
Compare
packages/core/fs/package.json
Outdated
@@ -48,6 48,8 @@ | |||
"check-ts": "tsc --noEmit index.d.ts" | |||
}, | |||
"dependencies": { | |||
"@parcel/feature-flags": "2.12.0", | |||
"@parcel/watcher-watchman-js": "2.12.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How can we make this dependency opt-in? Peer dep? try...catch around a require?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The package is so small (less than 50KB) that I wasn't going to worry about it. I can try make it opt in though.
Added a commit over v2...jondlm:parcel:jdlm-watchman-drop-in-replacement
Migrated to typescript but need to write some tests and integrate with the rest of the codebase.The ignore patterns section likely is one thing that needs to be changed carefully though we could support only a subset of the current ignore options and delegate to watchman.Update
Ignore support added. Right now globs and directories are supported.