Conversation
| @@ -0,0 +1,29 @@ | |||
| name: test-slack-build-result | |||
There was a problem hiding this comment.
maybe notify-result rather than build-result
Pull request was converted to draft
…t-actions into feature/notify-build * 'feature/notify-build' of https://github.com/elastic/oblt-actions: Update slack/notify-result/README.md
Co-authored-by: Jan Calanog <jan.calanog@elastic.co>
|
Will the ref show |
|
IMO, the explicit status text would be beneficial for color-blind people, especially green/red color-blindness. P.S. it could also be some kind of icon. |
I see, how can we ensure the size of the message box smaller and also support the color being explicit? |
Same behaviour as it's already done in the existing one: |
let's discuss this in a follow-up |
A more straightforward implementation of https://github.com/elastic/apm-pipeline-library/tree/0b1a309a85517985492e5d79572b13d59648e50e/.github/actions/notify-build-status