-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
ci: Add Slack notifications when releases and pre-releases are cut #9596
Conversation
Signed-off-by: Thane Thomson <connect@thanethomson.com>
@@ -8,7 +8,7 @@ on: | |||
- "v[0-9]+.[0-9]+.[0-9]+-rc[0-9]+" # e.g. v0.37.0-rc1, v0.38.0-rc10 | |||
|
|||
jobs: | |||
goreleaser: | |||
prerelease: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does this change imply we need to rename as well in lines 24 and 34 below? Or is it just a name collision?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, the change here just changes the job ID (like a variable name). The references to goreleaser on the other lines are important to keep as-is because they indicate the details of the GitHub action to execute.
"type": "section", | ||
"text": { | ||
"type": "mrkdwn", | ||
"text": ":sparkles: New Tendermint pre-release: <${{ env.RELEASE_URL }}|${{ github.ref_name }}>" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you also add :champagne:
?
... just kidding 😄
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually that's a good idea, but it'd be better for the releases (instead of the pre-releases) 😁
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very true :-)
We should probably steal that :) |
Automatically notify the team when pre-releases and releases are cut.
Pre-release notification rendered
Release notification rendered
PR checklist
CHANGELOG_PENDING.md
updated, or no changelog entry neededdocs/
) and code comments, or nodocumentation updates needed