Skip to content
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

chore: update actions to use google-github-actions/setup-gcloud@v1 #16674

Closed
wants to merge 7 commits into from
Closed

chore: update actions to use google-github-actions/setup-gcloud@v1 #16674

wants to merge 7 commits into from

Conversation

iuioiua
Copy link
Contributor

@iuioiua iuioiua commented Nov 16, 2022

This PR upgrades google-github-actions/setup-gcloud@v0 to the recommended google-github-actions/setup-gcloud@v1 within the build, publish-canary and update-dl-version jobs.

@iuioiua iuioiua marked this pull request as ready for review November 20, 2022 07:56
Copy link
Member

@bartlomieju bartlomieju left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are there no changes required with the version upgrade?

@iuioiua
Copy link
Contributor Author

iuioiua commented Nov 20, 2022

Your question prompted me to double-check. Now, google-github-actions/auth@v1 is used before setup. Unfortunately, there's no way to test these changes until a release is made.

@bartlomieju bartlomieju self-requested a review November 24, 2022 01:29
@bartlomieju
Copy link
Member

Note to self: if landed, apply these changes to #14448

startsWith(github.ref, 'refs/tags/'))
uses: google-github-actions/auth@v1
with:
project_id: denoland
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Where did you take that project_id from @iuioiua?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nvm, I see below

@bartlomieju
Copy link
Member

Thanks, I included your changes in #17358, gonna close this one.

@iuioiua iuioiua deleted the update-setup-gcloud-action branch January 12, 2023 00:19
bartlomieju added a commit that referenced this pull request Jan 12, 2023
…de (#17358)

This commit rolls up following PRs into a single one:
- #16672
- #16674
- #16972

Generation script was updated to take into account these changes.

Co-authored-by: Asher Gomez <ashersaupingomez@gmail.com>
Co-authored-by: Geert-Jan Zwiers <geertjanzwiers@protonmail.com>
Co-authored-by: David Sherret <dsherret@gmail.com>
dsherret added a commit that referenced this pull request Jan 13, 2023
…de (#17358)

This commit rolls up following PRs into a single one:
- #16672
- #16674
- #16972

Generation script was updated to take into account these changes.

Co-authored-by: Asher Gomez <ashersaupingomez@gmail.com>
Co-authored-by: Geert-Jan Zwiers <geertjanzwiers@protonmail.com>
Co-authored-by: David Sherret <dsherret@gmail.com>
dsherret added a commit that referenced this pull request Jan 13, 2023
…de (#17358)

This commit rolls up following PRs into a single one:
- #16672
- #16674
- #16972

Generation script was updated to take into account these changes.

Co-authored-by: Asher Gomez <ashersaupingomez@gmail.com>
Co-authored-by: Geert-Jan Zwiers <geertjanzwiers@protonmail.com>
Co-authored-by: David Sherret <dsherret@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants