-
Notifications
You must be signed in to change notification settings - Fork 13
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
feat: Add code #1
Conversation
f051843
to
7781e8d
Compare
I'm not sure why component installation works when this library is embedded in an action versus these tests. Example averikitsch/deploy-cloudrun#16 |
@averikitsch I tried these locally and beta seemed to pass for me. For gsutil I had to bump the gcloud version, maybe an issue with json parsing in that specific version? |
Code is ready for review. I will do another review of the documentation and workflows, then plan for publishing to NPM. |
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.
Overall LGTM
Some small nits below
@@ -0,0 +1,66 @@ | |||
@google-github-actions/setup-cloud-sdk / [Exports](modules.md) | |||
|
|||
# [Google GitHub actions: Setup Cloud SDK Client](https://github.com/google-github-actions) |
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 looks the same as the root readme?
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.
I am generating docs and it copies the README into the docs directory
|
||
## Table of contents | ||
|
||
### Functions |
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.
Is this autogenerated?
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.
I am generating this using typedoc (npm run docs
)
Will follow up on automating NPM publishing