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 optional credentials_file_path input for setup-gcloud #153
Add optional credentials_file_path input for setup-gcloud #153
Conversation
d055c05
to
515e778
Compare
515e778
to
8307ceb
Compare
@sethvargo How can I get this reviewed and merged? |
Hi @bricker, I'll review this shortly. I want to note that we are working on a new process for merging external contributions. |
@@ -34,7 +34,7 @@ module.exports = | |||
/******/ // the startup function |
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.
please remove the dist
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.
This file is built and added in a pre-commit hook (via Husky). Do you want me to delete this file, or just revert my changes? And should I remove the git hook too?
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.
The pre-commit hook has been removed. Please revert the changes to the dist/index.js
. We will be generating this file as a part of our release process.
Co-authored-by: Averi Kitsch <akitsch@google.com>
merging into a testing branch |
#213) * Add optional credentials_file_path input for setup-gcloud (#153) * Add optional credentials_file_path input to setup-gcloud * Update setup-gcloud/tests/setup-gcloud.test.ts Co-authored-by: Averi Kitsch <akitsch@google.com> Co-authored-by: Averi Kitsch <akitsch@google.com> * Respond to comments * lint * update timeout Co-authored-by: Bryan Ricker <bryancricker@gmail.com>
Fixes GH-144
setup-gcloud
which allows the caller to override the path at which the credentials are saved on disk. If not provided, the previous behavior (usingGITHUB_WORKSPACE
) is maintained.