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 some premature reloads with chokidar awaitWriteFinish #114

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

Conversation

evrys
Copy link

@evrys evrys commented Dec 9, 2021

Fixes some cases of #34 where miniflare reloads the worker js file as it is being written to (e.g. by webpack), rather than when it has finished emitting. Seems to particularly be a problem on Windows.

Fixes some cases of cloudflare#34 where miniflare reloads the worker
js file as it is being written to (e.g. by webpack), rather
than when it has finished emitting. Seems to particularly be
a problem on Windows.
@mrbbot
Copy link
Contributor

mrbbot commented Dec 10, 2021

Hey! 👋 Thanks for the PR! 🙂

Unfortunately, I haven't had time to maintain Miniflare 1, so it's kinda in limbo...

However, I've been working on Miniflare 2 which is hopefully going to be released very soon. You can try out a pre-release with npm i miniflare@next, find the full changelog here, and migration guide here.

@mrbbot mrbbot added fixed-in-next Fixed in next release legacy-1 Issue in Miniflare 1 labels Jan 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fixed-in-next Fixed in next release legacy-1 Issue in Miniflare 1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants