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.

Sign up for GitHub

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

Jump to bottom

Better handling for multiple watchers. #6

Merged

pacexy merged 1 commit into pacexy:main from s-KaiNet:main

Dec 19, 2021

Contributor

s-KaiNet commented Dec 12, 2021 •

edited

This PR adds a few improvements:

on every page refresh a new connection is sent, as a result, it creates multiple unnecessary file watchers, which is a waste of resources. Now on socket close event it destroys the corresponding watcher
added a file path to the output to see the actual file that triggered the reload.


          Better handling for multiple watchers.

21c803f

s-KaiNet force-pushed the main branch from 7eb53eb to 21c803f Compare

December 13, 2021 07:40

Owner

pacexy commented Dec 19, 2021

LGTM

pacexy merged commit 345cfec into pacexy:main

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment