Changeset 59693

Timestamp:

01/23/2025 05:12:02 PM (21 months ago)

Author:

johnbillion

Message:

Build/Test Tools: Correct the usage of the conclusion of the previous run within the Slack notifications workflow.

Prior to r59679 this value was echoed to GitHub output which meant that its surrounding double quotes lost their significance. Now this value is used directly in the job output it needs to be treated as a plain string.

This concludes the conclusion confusion.

See #82221

File:

• trunk/.github/workflows/slack-notifications.yml (modified) (1 diff)

trunk/.github/workflows/slack-notifications.yml

@@ -65 +65 @@
           retries: 2
           retry-exempt-status-codes: 418
+          result-encoding: string
           script: |
             const workflow_run = await github.rest.actions.getWorkflowRun({