-
Notifications
You must be signed in to change notification settings - Fork 1
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
Create initial chokidar file watcher #1
Conversation
BTW. Something that is not great is that a new process is started for every single watcher that is created. It should be possible to handle all watchers within a single process. Just would need to implement communication through stdin which I'm always dreading... |
For the record the updated implementation uses a single process for handling all watcher registrations. No matter how many language servers use watchers. The possible downside is that this could more easily reach the limit of open file descriptors but we'll see... |
Depends on sublimelsp/LSP#1764