-
-
Notifications
You must be signed in to change notification settings - Fork 8
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
Work around write-file-atomic Windows EPERM issue #15
Conversation
@coreyfarrell this is definitely a little icky 😄 I'll 👍, but is there a tracking issue on the Node.js project for this regression? |
To my knowledge it's not a node.js regression. See npm/write-file-atomic#28. |
@JaKXz I'm leaving the @ehmicky please point your package.json directly to 1689877 (the commit merged to |
Thanks for checking it! I've changed the reference to point to the commit hash, you can now safely remove the |
Fixes #14
Going to wait for more feedback from @ehmicky to verify this resolves the issue before I consider merging. Just posting the PR so it becomes visible to others in case someone objects to using a retry cycle. My thinking is that caching-transform success implies that the cache file has been successfully created.