Changeset 51535 for trunk/.github/workflows/phpunit-tests.yml

Timestamp:

08/03/2021 01:45:02 PM (4 years ago)

Author:

desrosj

Message:

Build/Test Tools: Expand Slack notifications for GitHub Actions.

This expands Slack notifications to include success, cancelled, and “fixed” GitHub Action workflow run outcomes in addition to failures.

A “fixed” outcome occurs when the previous run for a workflow failed and the current one succeeds. This matches the behavior that was native to TravisCI by setting on_success for notifications to change.

The message details and where each outcome is posted is controlled by Slack workflows.

The Slack notification logic has also been pulled into a separate workflow to prevent repeating code in every workflow.

See #52644.

File:

• trunk/.github/workflows/phpunit-tests.yml (modified) (1 diff)

trunk/.github/workflows/phpunit-tests.yml

@@ -237 +237 @@
           WPT_REPORT_API_KEY: "${{ secrets.WPT_REPORT_API_KEY }}"
         run: docker-compose run --rm -e WPT_REPORT_API_KEY -e WPT_PREPARE_DIR=/var/www -e WPT_TEST_DIR=/var/www php php test-runner/report.php
-
-  # Post workflow related status updates to Slack.
-  #
-  # When a job in this workflow fails, a message is posted to #core.
-  #
-  # This job should always require all other jobs in this workflow to complete before running.
-  slack-notifications:
-    name: Slack Notifications
-    runs-on: ubuntu-latest
-    needs: [ test-php ]
-    if: ${{ failure() && github.repository == 'WordPress/wordpress-develop' && github.event_name != 'pull_request' }}
-
-    steps:
-      - name: Post failure notification to Slack
-        uses: slackapi/slack-github-action@d5d276d7ae0f38f29322b80da9baf985cc80f8b1 # v1.15.0
-        with:
-          payload: "{\"workflow_name\":\"${{ github.workflow }}\",\"run_url\":\"https://github.com/${{ github.repository }}/actions/runs/${{ github.run_id }}\"}"
-        env:
-          SLACK_WEBHOOK_URL: ${{ secrets.SLACK_GHA_WEBHOOK_URL }}