Make WordPress Core


Ignore:
Timestamp:
10/25/2021 08:26:45 PM (2 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:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/.github/workflows/test-npm.yml

    r51924 r51937  
    160160    uses: WordPress/wordpress-develop/.github/workflows/slack-notifications.yml@master
    161161    needs: [ test-npm, test-npm-macos ]
    162     if: ${{ always() }}
     162    if: ${{ github.repository == 'WordPress/wordpress-develop' && github.event_name != 'pull_request' && always() }}
    163163    secrets:
    164164      SLACK_GHA_SUCCESS_WEBHOOK: ${{ secrets.SLACK_GHA_SUCCESS_WEBHOOK }}
Note: See TracChangeset for help on using the changeset viewer.