From 9c3733358a85b2716c9d2d4597293c9f29847e3f Mon Sep 17 00:00:00 2001 From: Riccardo Cipolleschi Date: Tue, 21 Jul 2026 16:40:23 +0100 Subject: [PATCH] Fix post-release workflow failures --- .../verifyArtifactsAreOnMaven.js | 6 +++--- .github/workflows/bump-podfile-lock.yml | 2 +- ...load-release-assets-for-dotslash-test.js.snap | 8 ++++---- .../upload-release-assets-for-dotslash.js | 16 ++++++++++------ 4 files changed, 18 insertions(+), 14 deletions(-) diff --git a/.github/workflow-scripts/verifyArtifactsAreOnMaven.js b/.github/workflow-scripts/verifyArtifactsAreOnMaven.js index bb76721ed219..5feb3777c032 100644 --- a/.github/workflow-scripts/verifyArtifactsAreOnMaven.js +++ b/.github/workflow-scripts/verifyArtifactsAreOnMaven.js @@ -34,9 +34,9 @@ const ARTIFACT_CLASSIFIERS = [ ]; async function verifyArtifactsAreOnMaven( - version: string, - retries: number = MAX_RETRIES, -): Promise { + version /*: string */, + retries /*: number */ = MAX_RETRIES, +) /*: Promise */ { if (version.startsWith('v')) { version = version.substring(1); } diff --git a/.github/workflows/bump-podfile-lock.yml b/.github/workflows/bump-podfile-lock.yml index 94895d94e19b..9e5c42764430 100644 --- a/.github/workflows/bump-podfile-lock.yml +++ b/.github/workflows/bump-podfile-lock.yml @@ -5,7 +5,7 @@ on: jobs: bump-podfile-lock: - runs-on: macos-latest + runs-on: macos-15 steps: - name: Checkout uses: actions/checkout@v6 diff --git a/scripts/releases/__tests__/__snapshots__/upload-release-assets-for-dotslash-test.js.snap b/scripts/releases/__tests__/__snapshots__/upload-release-assets-for-dotslash-test.js.snap index 210fcdc260df..0b52b3ef691e 100644 --- a/scripts/releases/__tests__/__snapshots__/upload-release-assets-for-dotslash-test.js.snap +++ b/scripts/releases/__tests__/__snapshots__/upload-release-assets-for-dotslash-test.js.snap @@ -28,7 +28,7 @@ Array [ Array [ Object { "asset_id": "1", - "owner": "facebook", + "owner": "react", "repo": "react-native", }, ], @@ -47,7 +47,7 @@ Array [ "content-type": "text/plain", }, "name": "test.tar.gz", - "owner": "facebook", + "owner": "react", "release_id": "1", "repo": "react-native", }, @@ -114,7 +114,7 @@ Array [ "content-type": "text/plain", }, "name": "test.tar.gz", - "owner": "facebook", + "owner": "react", "release_id": "1", "repo": "react-native", }, @@ -194,7 +194,7 @@ Array [ "content-type": "text/plain", }, "name": "test.tar.gz", - "owner": "facebook", + "owner": "react", "release_id": "1", "repo": "react-native", }, diff --git a/scripts/releases/upload-release-assets-for-dotslash.js b/scripts/releases/upload-release-assets-for-dotslash.js index d8e3441e6a95..cd808d9fff70 100644 --- a/scripts/releases/upload-release-assets-for-dotslash.js +++ b/scripts/releases/upload-release-assets-for-dotslash.js @@ -25,6 +25,10 @@ const nullthrows = require('nullthrows'); const path = require('path'); const {parseArgs} = require('util'); +// Upload POSTs are not preserved when GitHub redirects from the former owner. +const GITHUB_OWNER = 'react'; +const GITHUB_REPO = 'react-native'; + /*:: import type {DotSlashProvider, DotSlashHttpProvider, DotSlashArtifactInfo} from './utils/dotslash-utils'; import type {IOctokit} from './utils/octokit-utils'; @@ -138,8 +142,8 @@ async function getReleaseAssetMap( octokit /*: IOctokit */, ) /*: Promise */ { const existingAssets = await octokit.repos.listReleaseAssets({ - owner: 'facebook', - repo: 'react-native', + owner: GITHUB_OWNER, + repo: GITHUB_REPO, release_id: releaseId, }); return new Map(existingAssets.data.map(asset => [asset.name, asset])); @@ -305,8 +309,8 @@ async function maybeDeleteExistingReleaseAsset( } console.log(`[${name}] Deleting existing release asset...`); await octokit.repos.deleteReleaseAsset({ - owner: 'facebook', - repo: 'react-native', + owner: GITHUB_OWNER, + repo: GITHUB_REPO, asset_id: existingAsset.id, }); } @@ -360,8 +364,8 @@ async function uploadAndVerifyReleaseAsset( const { data: {browser_download_url}, } = await octokit.repos.uploadReleaseAsset({ - owner: 'facebook', - repo: 'react-native', + owner: GITHUB_OWNER, + repo: GITHUB_REPO, release_id: releaseId, name, data,