-
Notifications
You must be signed in to change notification settings - Fork 32
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
Add arch to cache key #72
base: main
Are you sure you want to change the base?
Conversation
Hmm... even telling it to use this commit in my workflow I'm still getting the x86-64 version installed. I'll keep investigating. |
Well, I guess it isn't actually using this commit after all. With debug logging turned on, the arch still isn't present in the cache key. So this might be the fix after all, I just need to make sure I'm actually using it in my testing. Maybe if I point it at my fork instead. |
It helps if I point it at a commit where I've updated the compiled JS in |
It also seems like the |
OK, I had to manually delete the cache dir Want me to open an issue about |
The |
@DeLaGuardo Any thoughts on this and #74? |
I was still seeing the x86-64 version installed in my self-hosted arm64 runner when using version 9.5, even after destroying and recreating it. So it seemed like a caching thing.