Skip to content

Add missing project_id input #50

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

Merged
merged 4 commits into from
Jan 17, 2024

Conversation

aelawson
Copy link
Contributor

@aelawson aelawson commented Jan 4, 2024

This GitHub action is missing the common project_id input - I ran into this while trying to setup a deployment workflow on one of my repositories.

I added the missing input and then updated the README to reflect it.

@aelawson aelawson requested a review from henrybell as a code owner January 4, 2024 21:49
Copy link

google-cla bot commented Jan 4, 2024

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@aelawson
Copy link
Contributor Author

aelawson commented Jan 4, 2024

Also no matter what I do I get a server error when trying to sign the CLA... not sure what's going on there.

@sethvargo
Copy link
Member

Also no matter what I do I get a server error when trying to sign the CLA... not sure what's going on there.

What error are you getting?

@aelawson aelawson force-pushed the add-project-id-input branch from b97b096 to 361a591 Compare January 5, 2024 01:15
@aelawson
Copy link
Contributor Author

aelawson commented Jan 5, 2024

Well I tried again and it worked. Not sure what happened.

@henrybell henrybell self-assigned this Jan 5, 2024
@henrybell
Copy link
Contributor

Hi @aelawson - thanks for the PR! I added some additional commits that include addition of the new input parameter to the GitHub Actions config, and a unit test. Before merging I was wondering if you'd be willing/able to describe your use-case -- at a guess I imagine that you are authenticating to deploy with a service account in one project that has permissions to create Cloud Deploy releases in another, and so need to specify project_id explicitly at release creation time? Thanks!

@sergiosalvatore
Copy link

@henrybell while I am not the author of this PR, I can confirm that your description is exactly the use case I need this option for.

@henrybell
Copy link
Contributor

Thanks @sergiosalvatore -- once merged, it'd be great if you could please let us know if there are any other outstanding features you're keen to see. Appreciate the input, and thanks again!

@sethvargo sethvargo merged commit 8424957 into google-github-actions:main Jan 17, 2024
sethvargo pushed a commit that referenced this pull request Jan 23, 2024
## What's Changed
* Update README and CI to use latest version by @sethvargo in
#49
* Add missing project_id input by @aelawson in
#50
* Update deps by @sethvargo in
#51

## New Contributors
* @aelawson made their first contribution in
#50

**Full Changelog**:
v1.0.0...21fa1d2
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy