Skip to content
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

fix: consider atime == mtime to be a non-change #1335

Closed
wants to merge 1 commit into from

Conversation

43081j
Copy link
Collaborator

@43081j 43081j commented Jun 30, 2024

Changes the file listener to treat atime === mtime as a non-change.

Currently, if these are equal, we fire a CHANGE event which causes bugs on windows (as windows emits two change events when you first write).

Changes the file listener to treat `atime === mtime` as a non-change.

Currently, if these are equal, we fire a `CHANGE` event which causes
bugs on windows (as windows emits two change events when you first
write).
@43081j
Copy link
Collaborator Author

43081j commented Jun 30, 2024

nevermind, looks like it breaks ubuntu now and fixes windows... 🤦‍♂️

a problem for another day

@43081j 43081j changed the base branch from v4 to master July 1, 2024 21:56
@43081j
Copy link
Collaborator Author

43081j commented Jul 1, 2024

saw you merged the v4 branch @paulmillr , awesome 🎉

ill try pick this one up tomorrow. if we can fix this bug we're pretty much ready to go i think

i could do with help on it though, especially given ill be out travelling from weds so my time will drop for a bit

@paulmillr paulmillr closed this Sep 13, 2024
@43081j 43081j deleted the at-mt branch September 13, 2024 17:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants