Changeset 57210 for trunk/.github/workflows/pull-request-comments.yml

Timestamp:

12/20/2023 02:50:11 PM (15 months ago)

Author:

swissspidy

Message:

Build/Test Tools: Post message for testing on Playground only after build succeeds.

Uses the workflow_run trigger to only leave pull request comments after the build jobs finish.

Props zieladam, desrosj.
See #59416.

File:

• trunk/.github/workflows/pull-request-comments.yml (modified) (4 diffs)

trunk/.github/workflows/pull-request-comments.yml

@@ -5 +5 @@
   pull_request_target:
     types: [ 'opened' ]
-    branches:
-      - trunk
+  workflow_run:
+    workflows: [ 'Test Build Processes' ]
+    types:
+      - completed
 
 # Cancels all previous workflow runs for pull requests that have not completed.
@@ -28 +30 @@
     if: ${{ github.repository == 'WordPress/wordpress-develop' && github.event_name == 'pull_request_target' }}
     steps:
-      - uses: wow-actions/welcome@72817eb31cda1de60f51893d80e2e82ce57f7e76 # v1.3.0
+      - name: Post a welcome comment
+        uses: wow-actions/welcome@72817eb31cda1de60f51893d80e2e82ce57f7e76 # v1.3.0
         with:
           FIRST_PR_REACTIONS: 'hooray'
@@ -82 +85 @@
       issues: write
       pull-requests: write
-    if: ${{ github.repository == 'WordPress/wordpress-develop' && github.event_name == 'pull_request_target' }}
+    if: >
+      github.repository == 'WordPress/wordpress-develop' &&
+      github.event.workflow_run.event == 'pull_request' &&
+      github.event.workflow_run.conclusion == 'success'
     steps:
-      - uses: actions/github-script@60a0d83039c74a4aee543508d2ffcb1c3799cdea # v7.0.1
+      - name: Download artifact
+        uses: actions/github-script@60a0d83039c74a4aee543508d2ffcb1c3799cdea # v7.0.1
         with:
           script: |
+            const artifacts = await github.rest.actions.listWorkflowRunArtifacts( {
+               owner: context.repo.owner,
+               repo: context.repo.repo,
+               run_id: ${{ github.event.workflow_run.id }},
+            } );
+
+            const matchArtifact = artifacts.data.artifacts.filter( ( artifact ) => {
+              return artifact.name === 'pr-number'
+            } )[0];
+
+            if ( ! matchArtifact ) {
+              core.setFailed( 'No artifact found!' );
+              return;
+            }
+
+            const download = await github.rest.actions.downloadArtifact( {
+               owner: context.repo.owner,
+               repo: context.repo.repo,
+               artifact_id: matchArtifact.id,
+               archive_format: 'zip',
+            } );
+
+            const fs = require( 'fs' );
+            fs.writeFileSync( '${{github.workspace}}/pr-number.zip', Buffer.from( download.data ) )
+
+      - name: Unzip the artifact containing the PR number
+        run: unzip pr-number.zip
+
+      - name: Leave a comment about testing with Playground
+        uses: actions/github-script@60a0d83039c74a4aee543508d2ffcb1c3799cdea # v7.0.1
+        with:
+          script: |
+            const fs = require( 'fs' );
+            const issue_number = Number( fs.readFileSync( './NR' ) );
+
             // Comments are only added after the first successful build. Check for the presence of a comment and bail early.
             const commentInfo = {
               owner: context.repo.owner,
               repo: context.repo.repo,
-              issue_number: ${{ github.event.number }}
+              issue_number,
             };
+
             const comments = ( await github.rest.issues.listComments( commentInfo ) ).data;
 
@@ -117 +160 @@
             For more details about these limitations and more, check out the [Limitations page](https://wordpress.github.io/wordpress-playground/limitations/) in the WordPress Playground documentation.
 
-            [Test this pull request with WordPress Playground](https://playground.wordpress.net/wordpress.html?pr=${{ github.event.number }}).
+            [Test this pull request with WordPress Playground](https://playground.wordpress.net/wordpress.html?pr=${ issue_number }).
             `;