Skip to content

Commit 145b7e8

Browse files
committed
Only log adding overrides when adding them
1 parent 6523d91 commit 145b7e8

File tree

1 file changed

+10
-3
lines changed

1 file changed

+10
-3
lines changed

src/commands/optimize/add-overrides.mts

Lines changed: 10 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -107,11 +107,8 @@ export async function addOverrides(
107107
)
108108
}
109109

110-
spinner?.setText(`Adding overrides to ${workspace}...`)
111-
112110
const depAliasMap = new Map<string, string>()
113111
const depEntries = getDependencyEntries(pkgEnvDetails)
114-
115112
const manifestEntries = manifestNpmOverrides.filter(({ 1: data }) =>
116113
semver.satisfies(
117114
// Roughly check Node range as semver.coerce will strip leading
@@ -121,6 +118,8 @@ export async function addOverrides(
121118
),
122119
)
123120

121+
let loggedAddingText = false
122+
124123
// Chunk package names to process them in parallel 3 at a time.
125124
await pEach(manifestEntries, 3, async ({ 1: data }) => {
126125
const { name: sockRegPkgName, package: origPkgName, version } = data
@@ -158,6 +157,10 @@ export async function addOverrides(
158157
if (!isWorkspaceRoot) {
159158
state.addedInWorkspaces.add(workspace)
160159
}
160+
if (!loggedAddingText) {
161+
spinner?.setText(`Adding overrides to ${workspace}...`)
162+
loggedAddingText = true
163+
}
161164
}
162165
depAliasMap.set(origPkgName, thisSpec)
163166
}
@@ -227,6 +230,10 @@ export async function addOverrides(
227230
overrides[origPkgName] = newSpec
228231
const addedOrUpdated = overrideExists ? 'updated' : 'added'
229232
state[addedOrUpdated].add(sockRegPkgName)
233+
if (!loggedAddingText) {
234+
spinner?.setText(`Adding overrides to ${workspace}...`)
235+
loggedAddingText = true
236+
}
230237
}
231238
}
232239
})

0 commit comments

Comments
 (0)