-
Notifications
You must be signed in to change notification settings - Fork 83
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: support cache computation on windows #157
Conversation
Thank you for this @GerkinDev,
|
@KyleTryon fixed. |
Your development orb has been published. It will expire in 30 days. |
Your development orb has been published. It will expire in 30 days. |
On windows runners, the cache keys fails to be computed because of the `checksum` function that can't find the file in /tmp. Indeed, it seems that bash `/tmp` is not the runner's `C:\tmp` directory. See https://app.circleci.com/pipelines/github/KnodesCommunity/typedoc-plugins/693/workflows/897cb674-230e-46ae-99f5-b10d93f4b723/jobs/3872/parallel-runs/0/steps/0-106 for an example of failed step
f000823
to
ad27e8b
Compare
ad27e8b
to
4a4fca6
Compare
Thanks, @GerkinDev! Your changes are working great with the existing macOS and Linux tests. I included one for Windows here: 4a4fca6. |
Glad I can count on caching now :) thank you ! |
Not sure if I'm missing something, but dependency caching doesn't seem to work for me on Windows runners (latest version, just running I know Windows isn't officially supported, but I'm curious how people are using this successfully. |
On windows runners, the cache keys fails to be computed because of the
checksum
function that can't find the file in /tmp. Indeed, it seems that bash/tmp
is not the runner'sC:\tmp
directory.See https://app.circleci.com/pipelines/github/KnodesCommunity/typedoc-plugins/693/workflows/897cb674-230e-46ae-99f5-b10d93f4b723/jobs/3872/parallel-runs/0/steps/0-106 for an example of failed step