Skip to content

refactor: migrate to goreleaser native Cloudsmith support#1046

Merged
Soner (shyim) merged 1 commit into
mainfrom
refactor/migrate-to-goreleaser-cloudsmith
May 21, 2026
Merged

refactor: migrate to goreleaser native Cloudsmith support#1046
Soner (shyim) merged 1 commit into
mainfrom
refactor/migrate-to-goreleaser-cloudsmith

Conversation

@shyim
Copy link
Copy Markdown
Member

Summary

  • Replace manual cloudsmith-cli pip install and push steps in release.yml with goreleaser's built-in cloudsmiths configuration in .goreleaser.yaml
  • This removes the dependency on the cloudsmith-cli Python package and lets goreleaser handle distribution uploads natively

Test plan

  • Verify goreleaser dry-run produces correct Cloudsmith upload config
  • Verify release pipeline pushes .deb, .rpm, and .apk packages to friendsofshopware/stable

💘 Generated with Crush

Replace manual cloudsmith-cli pip install and push steps in the release
workflow with goreleaser's built-in cloudsmiths configuration. This
simplifies the pipeline and moves package distribution logic into the
goreleaser config where it belongs.

💘 Generated with Crush

Assisted-by: Crush:deepseek-v4-flash
@shyim Soner (shyim) merged commit 00d917f into main May 21, 2026
2 checks passed
@shyim Soner (shyim) deleted the refactor/migrate-to-goreleaser-cloudsmith branch May 21, 2026 09:10
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 9cc11948ed

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "Codex (@codex) review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "Codex (@codex) address that feedback".

Comment thread .goreleaser.yaml
- organization: friendsofshopware
repository: stable
secret_name: CLOUDSMITH_API_KEY
disable: '{{ if .Prerelease }}true{{ end }}'
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Replace unsupported Cloudsmith disable key with skip

In the new cloudsmiths block, disable is not a valid Cloudsmith config field in GoReleaser (the Cloudsmith publisher docs list skip for conditional publishing), so this can break goreleaser release with a config unmarshal/validation error, or at minimum fail to gate prerelease uploads as intended. Because this runs inside .github/workflows/release.yml for every tag, a bad key here can block the entire release job instead of only Cloudsmith publishing.

Useful? React with 👍 / 👎.

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.

1 participant