chore: predicate slack notification job on other jobs #16106
Conversation
always() does not seem to work Signed-off-by: Danny Kopping <[email protected]>
bcb70a9 to
4b95ea4
Compare
| runs-on: ubuntu-latest | ||
| if: always() && failure() |
There was a problem hiding this comment.
always() && failure() is the same as failure(). I think the operative part here is the needs section?
There was a problem hiding this comment.
I mentioned the operative part in the title, although agree this could've been more explicit.
Clause Sonnet suggested that always() could be used if you wanted to handle all jobs (i.e. not have to exhaustively define needs), but maybe that's wrong.
I hate GHA.
|
Also, are you able to test it by manually specifying your branch? |
I'm busy with other things so didn't bother to check the semantics of the "We do this not because it's easy, but because we thought it'd be easy" |
|
Let's goooooo |
always()does not seem to workExtending #16105