-
Notifications
You must be signed in to change notification settings - Fork 4
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
[Feature] Add ability to watch modules #3
Conversation
62e03fb
to
b66cadb
Compare
b66cadb
to
b3da32b
Compare
Changes seem nice and simple 👍 |
// Or if you're using `yarn link` (or npm) to symlink external dependencies | ||
// during dev, pass in an array of modules to watch. Changes made internally | ||
// will be instantly available in the app. | ||
watchModules: [ |
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.
Is this config necessary? Could this module watch all node modules/symlinks by default or would that be too much?
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.
@craigspaeth - opt-in is necessary as watching the whole folder would be extremely slow :)
Awesome! Excited to have this help the Reaction <> Force disparity! 👏 Left a small question, but code LGTM—feel free to merge at will. |
🚀 PR was released in v1.4.1 🚀 |
This PR adds additional watching ability, making it easy to listen for changes in
node_modules
and automatically reload dependencies on request. This is useful when working across library boundaries whereyarn link
is often used to symlink NPM deps during development.Closes #2