From 72e48b8e16cab83e22b5cf24ac73499a48aba23c Mon Sep 17 00:00:00 2001 From: Andrew Clark Date: Wed, 8 Dec 2021 02:55:46 -0500 Subject: [PATCH] Fix: Don't skip writing updated package.json Another fix to previous commit. The special case for use-sync-external-store still needs to write out the updated package.json, because we also use that branch to update the version field. --- scripts/rollup/build-all-release-channels.js | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/scripts/rollup/build-all-release-channels.js b/scripts/rollup/build-all-release-channels.js index 0852b8af2e..1f31df0d0c 100644 --- a/scripts/rollup/build-all-release-channels.js +++ b/scripts/rollup/build-all-release-channels.js @@ -250,14 +250,14 @@ function updatePackageVersions( // we don't override to the latest version. We should figure out some // better way to handle this. // TODO: Remove this special case. - continue; - } - for (const dep of Object.keys(packageInfo.peerDependencies)) { - const depVersion = versionsMap.get(dep); - if (depVersion !== undefined) { - packageInfo.peerDependencies[dep] = pinToExactVersion - ? depVersion - : '^' + depVersion; + } else { + for (const dep of Object.keys(packageInfo.peerDependencies)) { + const depVersion = versionsMap.get(dep); + if (depVersion !== undefined) { + packageInfo.peerDependencies[dep] = pinToExactVersion + ? depVersion + : '^' + depVersion; + } } } }