Skip to content
This repository has been archived by the owner on Oct 1, 2020. It is now read-only.

Make file watching more robust #195

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

fvgs
Copy link

@fvgs fvgs commented Mar 1, 2017

Fixes #189, fixes #193

See #193 for more information

Use gaze instead of fs.watch() for file watching.

Fix an issue where fs.watch() does not detect file changes when using
certain text editor features such as backupcopy in Vim.

Fix an issue where multiple change events are received for the same
change.

Improve HMR speed by removing manual throttling on file changes in favor
of the more robust debouncing performed by gaze.

@paulcbetts mentioned experiencing issues on Windows in #190. It would be great to hear if this resolves those issues.

Use `gaze` instead of `fs.watch()` for file watching.

Fix an issue where `fs.watch()` does not detect file changes when using
certain text editor features such as `backupcopy` in Vim.

Fix an issue where multiple change events are received for the same
change.

Improve HMR speed by removing manual throttling on file changes in favor
of the more robust debouncing performed by `gaze`.
@fvgs
Copy link
Author

fvgs commented Aug 9, 2017

@paulcbetts Any input for moving this forward?

fasterthanlime pushed a commit to fasterthanlime/electron-compile that referenced this pull request Aug 24, 2017
fasterthanlime added a commit to fasterthanlime/electron-compile that referenced this pull request Aug 24, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

File watching does not work correctly with Vim watchPath calls subscribers multiple times
1 participant