Skip to content

ci: post #deploy Slack message on CLI release#194

Merged
zfarrell merged 1 commit into
mainfrom
ci/slack-release-notification
Jun 29, 2026
Merged

ci: post #deploy Slack message on CLI release#194
zfarrell merged 1 commit into
mainfrom
ci/slack-release-notification

Conversation

@zfarrell

Copy link
Copy Markdown
Contributor

Adds a Slack notification to the release workflow's announce job so CLI releases post the same clean 📦 hotdata-cli vX.Y.Z released — release notes message in #deploy that the SDKs use, instead of relying on the generic GitHub-app workflow notification. Reuses the shared SLACK_CI_BOT_TOKEN org secret (now scoped to this repo) and the #deploy channel.

@codecov

codecov Bot commented Jun 29, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@claude claude Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Clean, well-guarded change. Verified the Slack step only runs on real tag releases (host requires publishing == true, announce requires host success), continue-on-error keeps a published release from being marked failed, the JSON is built safely with jq --arg, the message backticks are escaped, the token is never echoed, and .ok is asserted to catch Slack's HTTP-200-on-error responses. LGTM.

@zfarrell zfarrell merged commit a2a3c38 into main Jun 29, 2026
14 checks passed
@zfarrell zfarrell deleted the ci/slack-release-notification branch June 29, 2026 22:17
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