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

fix: move goreleaser action workflow job to release workflow to get go binaries attached to release via automation #64

Merged
merged 1 commit into from
Jan 9, 2025

Conversation

jmeridth
Copy link
Member

@jmeridth jmeridth commented Jan 8, 2025

Due to security limitations when a workflow is run with GITHUB_TOKEN (our release job) another workflow that also uses GITHUB_TOKEN will not fire-off when in another workflow. I've moved goreleaser to the release workflow.

Due to security limitations when a workflow is run with GITHUB_TOKEN (our release job) another workflow that also uses GITHUB_TOKEN will not fire-off when in another workflow.  I've moved goreleaser to the release workflow.

Signed-off-by: jmeridth <[email protected]>
@jmeridth jmeridth self-assigned this Jan 8, 2025
@Copilot Copilot bot review requested due to automatic review settings January 8, 2025 15:34

Choose a reason for hiding this comment

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

Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.

Files not reviewed (1)
  • .github/workflows/goreleaser.yml: Language not supported

Tip: If you use Visual Studio Code, you can request a review from Copilot before you push from the "Source Control" tab. Learn more

@github-actions github-actions bot added the fix label Jan 8, 2025
@jmeridth jmeridth requested a review from a team January 8, 2025 19:40
@eddie-knight eddie-knight merged commit b521947 into main Jan 9, 2025
4 checks passed
@eddie-knight eddie-knight deleted the jm_move_go_releaser branch January 9, 2025 04:49
Copy link

@ASLE90 ASLE90 left a comment

Choose a reason for hiding this comment

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

The most

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants