Changeset 59217

Timestamp:

10/11/2024 07:02:30 PM (8 weeks ago)

Author:

joemcgill

Message:

Build/Test Tools: Fix Performance Test workflows on release branches.

This fixes an issue after [59170] that was causing the Performance Test workflows to fail on older branches since the CODEVITALS_PROJECT_TOKEN input value was marked as required but was not being passed.

This is a follow-up to [59214] and [59215].

Fixes #62153.
Props desrosj, joemcgill, flixos90, swissspidy.

File:

• trunk/.github/workflows/reusable-performance.yml (modified) (2 diffs)

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

@@ -30 +30 @@
       CODEVITALS_PROJECT_TOKEN:
         description: 'The authorization token for https://www.codevitals.run/project/wordpress.'
-        required: true
+        required: false
 
 env:
@@ -330 +330 @@
           CODEVITALS_PROJECT_TOKEN: ${{ secrets.CODEVITALS_PROJECT_TOKEN }}
           HOST_NAME: "www.codevitals.run"
-        run: node ./tests/performance/log-results.js $CODEVITALS_PROJECT_TOKEN trunk $GITHUB_SHA $BASE_SHA $COMMITTED_AT $HOST_NAME
+        run: |
+          if [ -z "$CODEVITALS_PROJECT_TOKEN" ]; then
+            echo "Performance results could not be published. 'CODEVITALS_PROJECT_TOKEN' is not set" >> $GITHUB_OUTPUT
+            exit 1
+          fi
+          node ./tests/performance/log-results.js $CODEVITALS_PROJECT_TOKEN trunk $GITHUB_SHA $BASE_SHA $COMMITTED_AT $HOST_NAME
 
       - name: Ensure version-controlled files are not modified or deleted