mirror/pnpm
1
0
Fork 0
mirror of https://github.com/pnpm/pnpm.git synced 2026-04-10 18:18:56 -04:00
Code Issues Packages Projects Releases Wiki Activity

fix: don't ignore the overrides field on partial install/update

Browse Source
This commit is contained in:
Zoltan Kochan
2020-10-31 14:12:19 +02:00
parent 23c735c00d
commit c4ec56eeb2
2 changed files with 11 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
.changeset/polite-houses-design.md Normal file
View File

@@ -0,0 +1,5 @@
---
"supi": patch
---
Don't ignore the "overrides" field when install/update doesn't include the root project.

9
packages/supi/src/install/index.ts
View File

@@ -137,12 +137,15 @@ export async function mutateModules (
const installsOnly = projects.every((project) => project.mutation === 'install')
opts['forceNewModules'] = installsOnly
const ctx = await getContext(projects, opts)
const rootProject = ctx.projects.find(({ id }) => id === '.')
const rootProjectManifest = ctx.projects.find(({ id }) => id === '.')?.manifest ??
// When running install/update on a subset of projects, the root project might not be included,
// so reading its manifest explicitly hear.
await safeReadProjectManifestOnly(opts.lockfileDir)
// We read Yarn's resolutions field for compatibility
// but we really replace the version specs to any other version spec, not only to exact versions,
// so we cannot call it resolutions
const overrides = rootProject
? rootProject.manifest.pnpm?.overrides ?? rootProject.manifest.resolutions
const overrides = rootProjectManifest
? rootProjectManifest.pnpm?.overrides ?? rootProjectManifest.resolutions
: undefined
if (!R.isEmpty(overrides ?? {})) {
const versionsOverrider = createVersionsOverrider(overrides!)