Skip to content

Commit 2b7483b

Browse files
craig[bot]RaduBerinde
andcommitted
Merge #148231
148231: github: fix payload in check-pebble-dep.yml r=RaduBerinde a=RaduBerinde Epic: none Release note: None I verified that the payload gets constructed correctly here: https://github.com/RaduBerinde/workflows/actions/runs/15619021112/job/43998983557 ~I don't know why the action didn't fail there though, since I used a bogus token.~ Ah, there is an `errors` option for that: https://github.com/RaduBerinde/workflows/actions/runs/15619124768/job/43999314443 Co-authored-by: Radu Berinde <[email protected]>
2 parents 7b71d93 + e21ceff commit 2b7483b

File tree

1 file changed

+7
-6
lines changed

1 file changed

+7
-6
lines changed

.github/workflows/check-pebble-dep.yml

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -31,13 +31,14 @@ jobs:
3131
if: steps.run_script.outputs.exitcode != '0'
3232
uses: slackapi/[email protected]
3333
with:
34+
errors: true
3435
method: chat.postMessage
3536
token: ${{ secrets.PEBBLE_SLACK_BOT_TOKEN }}
3637
# The channel ID is for #storage-notifications.
3738
payload: |
38-
channel: C08JE13CM9S
39-
text: |
40-
Some Pebble dependencies are not up to date. Details below:
41-
```
42-
${{ steps.run_script.outputs.output }}
43-
```
39+
{
40+
"channel": "C08JE13CM9S",
41+
"text": ${{ toJson(format(
42+
'Some Pebble dependencies are not up to date. Details below:\n```{0}```',
43+
steps.run_script.outputs.output)) }}
44+
}

0 commit comments

Comments
 (0)