Changeset 59582

Timestamp:

01/06/2025 09:19:51 PM (7 days ago)

Author:

flixos90

Message:

Build/Test Tools: Fix incorrect commit time being reported to WordPress Code Vitals Dashboard.

2nd attempt of [59570].

Props mukesh27, ayeshrajans, swissspidy, desrosj.
Fixes #62766.

Location:

trunk

Files:

• .github/workflows/reusable-performance.yml (modified) (2 diffs)
• tests/performance/log-results.js (modified) (2 diffs)

trunk/.github/workflows/reusable-performance.yml

@@ -332 +332 @@
         # Only needed when publishing results.
         if: ${{ github.event_name == 'push' && github.ref == 'refs/heads/trunk' && ! inputs.memcached && ! inputs.multisite }}
-        uses: actions/github-script@60a0d83039c74a4aee543508d2ffcb1c3799cdea # v7.0.1
-        id: commit-timestamp
-        with:
-          github-token: ${{ secrets.GITHUB_TOKEN }}
-          script: |
-            const commit_details = await github.rest.git.getCommit({ owner: context.repo.owner, repo: context.repo.repo, commit_sha: context.sha });
-            return parseInt((new Date( commit_details.data.author.date ).getTime() / 1000).toFixed(0))
+        # Write to an environment variable to have the output available in later steps of the job.
+        run: echo "COMMITTED_AT=$(git show -s $GITHUB_SHA --format='%cI')" >> $GITHUB_ENV
 
       - name: Publish performance results
@@ -345 +340 @@
         env:
           BASE_SHA: ${{ steps.base-sha.outputs.result }}
-          COMMITTED_AT: ${{ steps.commit-timestamp.outputs.result }}
           CODEVITALS_PROJECT_TOKEN: ${{ secrets.CODEVITALS_PROJECT_TOKEN }}
           HOST_NAME: "www.codevitals.run"

trunk/tests/performance/log-results.js

@@ -12 +12 @@
  */
 const https = require( 'https' );
-const [ token, branch, hash, baseHash, timestamp, host ] =
+const [ token, branch, hash, baseHash, date, host ] =
     process.argv.slice( 2 );
 const { median, parseFile, accumulateValues } = require( './utils' );
@@ -88 +88 @@
         hash,
         baseHash,
-        timestamp: parseInt( timestamp, 10 ),
+        timestamp: date,
         metrics: metrics,
         baseMetrics: baseMetrics,