Skip to content

feat: Add send-slack-notification action#42

Merged
NickLarsenNZ merged 8 commits intomainfrom
feat/send-slack-notification
May 9, 2025
Merged

feat: Add send-slack-notification action#42
NickLarsenNZ merged 8 commits intomainfrom
feat/send-slack-notification

Conversation

@Techassi
Copy link
Member

@Techassi Techassi commented May 8, 2025

This PR adds a new send-slack-notification action with support for threading. The action will create threads for each individual workflow run. This allows multiple attempts of the same run to be displayed as a thread. This is mainly used in docker-image workflows.

@Techassi Techassi self-assigned this May 8, 2025
@Techassi Techassi marked this pull request as ready for review May 8, 2025 13:39
@Techassi Techassi moved this to Development: Waiting for Review in Stackable Engineering May 8, 2025
@Techassi
Copy link
Member Author

Techassi commented May 8, 2025

I will create a follow-up PR which improves the thread ID storing and retrieval, as well as the option to specify if the notification should be marked as a failure or as a success.

@NickLarsenNZ NickLarsenNZ moved this from Development: Waiting for Review to Development: In Review in Stackable Engineering May 9, 2025
@NickLarsenNZ NickLarsenNZ self-requested a review May 9, 2025 10:34
NickLarsenNZ

This comment was marked as outdated.

Co-authored-by: Nick <10092581+NickLarsenNZ@users.noreply.github.com>
@Techassi Techassi requested a review from NickLarsenNZ May 9, 2025 10:36
Copy link
Member

@NickLarsenNZ NickLarsenNZ left a comment

Choose a reason for hiding this comment

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

LGTM

@NickLarsenNZ NickLarsenNZ merged commit b00c3b1 into main May 9, 2025
2 checks passed
@NickLarsenNZ NickLarsenNZ deleted the feat/send-slack-notification branch May 9, 2025 10:37
@Techassi Techassi moved this from Development: In Review to Development: Done in Stackable Engineering May 9, 2025
@lfrancke lfrancke moved this from Development: Done to Done in Stackable Engineering May 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants