Changeset 51937 for trunk/.github/workflows/test-coverage.yml

Timestamp:

10/25/2021 08:26:45 PM (3 years ago)

Author:

desrosj

Message:

Build/Test Tools: Use the correct workflow name in notifications on workflow_run.

When a workflow is triggered through a workflow_run event, the context is not the original workflow. The details about the original workflow are passed through the github.event context.

This also moves the conditional check controlling whether the Slack workflow is run into the calling workflows to prevent them from running for pull requests.

Follow up to [51921-51922,51924-51925,51934].

See #53363.

File:

• trunk/.github/workflows/test-coverage.yml (modified) (1 diff)

trunk/.github/workflows/test-coverage.yml

@@ -174 +174 @@
     uses: WordPress/wordpress-develop/.github/workflows/slack-notifications.yml@master
     needs: [ test-coverage-report ]
-    if: ${{ always() }}
+    if: ${{ github.repository == 'WordPress/wordpress-develop' && github.event_name != 'pull_request' && always() }}
     secrets:
       SLACK_GHA_SUCCESS_WEBHOOK: ${{ secrets.SLACK_GHA_SUCCESS_WEBHOOK }}