Skip to content

Commit a5602a3

Browse files
authored
Fix noisy Discord notifications on PR merge (#3650)
# Description of Changes Filter out the discord notification job itself, as well as any neutral results. # API and ABI breaking changes CI only. # Expected complexity level and risk 1 # Testing Unsure how to test this without merging the PR Co-authored-by: Zeke Foppa <[email protected]>
1 parent 61e167f commit a5602a3

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

.github/workflows/discord-posts.yml

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,10 @@ jobs:
3333
FAILED_CHECKS="$(
3434
gh pr checks "${{github.event.pull_request.html_url}}" \
3535
--json 'workflow,state,name' |
36-
jq '.[] | select(.state != "SUCCESS")' |
36+
jq '.[]
37+
| select(.workflow != "Discord notifications")
38+
| select(.state != "SUCCESS" and .state != "NEUTRAL")
39+
' |
3740
jq -r '"\(.workflow) / \(.name): \(.state)"'
3841
)"
3942
message+=$'\n'

0 commit comments

Comments
 (0)