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
Revert "Use RUNNER_TEMP to export credentials" #149
Conversation
This reverts commit 191a0f3.
Is there no other place these can be kept? Does It'd be nice if there was a full reference on the different env vars and how they work out in VM vs Container. |
It's how looks default run of Docker based GH action for our case: Maybe, the way to use one more input param for this action and to set either RUNNER_TEMP or GITHUB_WORKSPACE as a "workspace" also will be fine. |
Sure, but that's a breaking change. I think we should revert this. If we want to introduce that change, it should be in a new tagged version so people can upgrade safely. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Seems like macos tests are queued/failing but I dont think its related.
Reverts #148
As I kinda suspected but couldn't remember, there's a reason we use the action's temporary directory.
/cc @travisgroth @ViacheslavKudinov