Skip to content

Conversation

vcheung-stripe
Copy link
Collaborator

Reviewers

r? @suz-stripe
cc @stripe/developer-products

Summary

Context: After merging #884, I released v.1.10.1 to trigger the release action. The release action failed because it couldn't find the environment variable I was reading: https://github.com/stripe/stripe-cli/runs/6694623367?check_suite_focus=true. This PR will hopefully set the environment variable correctly. As always, not sure how to test other than to make a release, so I'll release v1.10.2 after this.

@suz-stripe
Copy link
Contributor

I believe the Goreleaser GitHub Action explicitly reads the GITHUB_TOKEN env var so I'm not sure that this solves the issue unfortunately. Bear in mind that it's been a while since I have touched this Action workflow, so I might be missing something here.

@suz-stripe
Copy link
Contributor

I just saw this: https://github.com/stripe/stripe-cli/pull/884/files
sorry! This LGTM.

@vcheung-stripe vcheung-stripe merged commit ba45b60 into stripe:master Jun 1, 2022
@vcheung-stripe vcheung-stripe deleted the vcheung/fix-release-action-2 branch June 1, 2022 17:56
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