From db40320063b8f7574fba96227d4b62792f767bdd Mon Sep 17 00:00:00 2001 From: Hugo Alliaume Date: Tue, 11 Feb 2025 08:22:18 +0100 Subject: [PATCH] [CI] Fix links to changed dist files --- .github/generate-dist-files-size-diff.mjs | 18 +++++++++--------- .github/workflows/dist-files-size-diff.yaml | 2 ++ 2 files changed, 11 insertions(+), 9 deletions(-) diff --git a/.github/generate-dist-files-size-diff.mjs b/.github/generate-dist-files-size-diff.mjs index 3f9022d1c4b..1bdc0a3d8ff 100644 --- a/.github/generate-dist-files-size-diff.mjs +++ b/.github/generate-dist-files-size-diff.mjs @@ -7,8 +7,8 @@ Usage: ```shell BASE_DIST_FILES='{"src/Autocomplete/assets/dist/controller.js":{"size":15382,"size_gz":3716},"src/Chartjs/assets/dist/controller.js":{"size":2281,"size_gz":771},"src/Cropperjs/assets/dist/controller.js":{"size":1044,"size_gz":475}}' \ PR_DIST_FILES='{"src/Chartjs/assets/dist/controller.js":{"size":1281,"size_gz":171},"src/Cropperjs/assets/dist/controller.js":{"size":1044,"size_gz":475},"src/Cropperjs/assets/dist/style.min.css":{"size":32,"size_gz":66},"src/Dropzone/assets/dist/controller.js":{"size":3199,"size_gz":816},"src/Map/src/Bridge/Google/assets/dist/foo.js":{"size":3199,"size_gz":816}}' \ -GITHUB_REPOSITORY='symfony/ux' \ -GITHUB_HEAD_REF='my-branch-name' \ +BASE_REPO_NAME='kocal/symfony-ux' \ +BASE_REF_NAME='my-branch-name' \ node .github/generate-dist-files-size-diff.mjs ``` */ @@ -21,12 +21,12 @@ if (!process.env.PR_DIST_FILES) { throw new Error('Missing or invalid "PR_DIST_FILES" env variable.'); } -if (!process.env.GITHUB_REPOSITORY) { - throw new Error('Missing or invalid "GITHUB_REPOSITORY" env variable.'); +if (!process.env.BASE_REPO_NAME) { + throw new Error('Missing or invalid "BASE_REPO_NAME" env variable.'); } -if (!process.env.GITHUB_HEAD_REF) { - throw new Error('Missing or invalid "GITHUB_HEAD_REF" env variable.'); +if (!process.env.BASE_REF_NAME) { + throw new Error('Missing or invalid "BASE_REF_NAME" env variable.'); } /** @@ -67,7 +67,7 @@ function formatDiffPercent(percent) { } export function main() { - const repoUrl = `https://github.com/${process.env.GITHUB_REPOSITORY}`; + const repoUrl = `https://github.com/${process.env.BASE_REPO_NAME}`; /** @type {Record} */ const base = JSON.parse(process.env.BASE_DIST_FILES); /** @type {Record} */ @@ -108,7 +108,7 @@ export function main() { meta: { packageName, bridgeName, - url: isBridge ? `${repoUrl}/tree/${process.env.GITHUB_HEAD_REF}/src/${packageName}/src/Bridge/${bridgeName}/assets/dist` : `${repoUrl}/tree/${process.env.GITHUB_HEAD_REF}/src/${packageName}/assets/dist`, + url: isBridge ? `${repoUrl}/tree/${process.env.BASE_REF_NAME}/src/${packageName}/src/Bridge/${bridgeName}/assets/dist` : `${repoUrl}/tree/${process.env.BASE_REF_NAME}/src/${packageName}/assets/dist`, }, files: new Set(), }); } @@ -126,7 +126,7 @@ export function main() { }, meta: { fileNameShort: file.replace(isBridge ? `src/${file.split('/')[1]}/src/Bridge/${file.split('/')[4]}/assets/dist/` : `src/${file.split('/')[1]}/assets/dist/`, ''), - fileNameUrl: `${repoUrl}/blob/${process.env.GITHUB_HEAD_REF}/${file}`, + fileNameUrl: `${repoUrl}/blob/${process.env.BASE_REF_NAME}/${file}`, }, }); } diff --git a/.github/workflows/dist-files-size-diff.yaml b/.github/workflows/dist-files-size-diff.yaml index ef80d89abc4..368f1bafd3b 100644 --- a/.github/workflows/dist-files-size-diff.yaml +++ b/.github/workflows/dist-files-size-diff.yaml @@ -50,6 +50,8 @@ jobs: env: BASE_DIST_FILES: ${{ steps.base-dist-files.outputs.files }} PR_DIST_FILES: ${{ steps.pr-dist-files.outputs.files }} + BASE_REPO_NAME: ${{ github.event.pull_request.base.repo.full_name }} + BASE_REF_NAME: ${{ github.event.pull_request.base.ref }} with: result-encoding: string script: |