Skip to content

Conversation

@joakimen
Copy link
Contributor

@joakimen joakimen commented Dec 16, 2025

Summary

Adds support for mentioning users and groups when pipelines fail.

Description

  • Add SlackMention class: Responsible for formatting users, groups and special mentions into the Slack API required format. Table-driven tests exercise valid/invalid format inputs.
  • First green->red only: Users are only mentioned on the first pipeline failure that occur, not subsequent ones.
  • Keep handler slim: Keep formatting and validation logic outside of lambda handler code, as handlers are harder to test than regular project code

TODO

  • Test functionality in sandbox environment

@joakimen joakimen marked this pull request as ready for review December 16, 2025 14:26
@joakimen joakimen requested a review from a team as a code owner December 16, 2025 14:26
@joakimen
Copy link
Contributor Author

@sirius mulighet for en review?

@joakimen joakimen force-pushed the cals-1199-add-slack-notification-mention-support branch from f4b9b70 to 67bc93e Compare December 16, 2025 14:55
@joakimen joakimen force-pushed the cals-1199-add-slack-notification-mention-support branch from 67bc93e to acc35ba Compare December 17, 2025 11:08
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.

2 participants