Skip to content

Commit d4b2046

Browse files
author
Caleb Barnes
authored
fix(config): auto install required extensions in buildbot (#6386)
* Removes check for 'accounts' because it is not available in buildbot context.
1 parent 93e0ab6 commit d4b2046

File tree

2 files changed

+1
-16
lines changed

2 files changed

+1
-16
lines changed

packages/config/src/main.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -172,7 +172,6 @@ export const resolveConfig = async function (opts): Promise<Config> {
172172

173173
const updatedIntegrations = await handleAutoInstallExtensions({
174174
featureFlags,
175-
accounts,
176175
integrations,
177176
siteId,
178177
accountId,

packages/config/src/utils/extensions/auto-install-extensions.ts

Lines changed: 1 addition & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,6 @@ interface AutoInstallOptions {
1818
accountId: string
1919
token: string
2020
cwd: string
21-
accounts: any
2221
integrations: IntegrationResponse[]
2322
offline: boolean
2423
testOpts: any
@@ -32,7 +31,6 @@ export async function handleAutoInstallExtensions({
3231
accountId,
3332
token,
3433
cwd,
35-
accounts,
3634
integrations,
3735
offline,
3836
testOpts = {},
@@ -94,18 +92,6 @@ export async function handleAutoInstallExtensions({
9492
}
9593

9694
try {
97-
const account = accounts?.find((account: any) => account.id === accountId)
98-
if (!account) {
99-
console.error(`Failed to auto install extension(s): Couldn't find 'account' with id '${accountId}'`, {
100-
accountId,
101-
siteId,
102-
cwd,
103-
offline,
104-
mode,
105-
})
106-
return integrations
107-
}
108-
10995
const packageJson = getPackageJSON(cwd)
11096
if (
11197
!packageJson?.dependencies ||
@@ -126,7 +112,7 @@ export async function handleAutoInstallExtensions({
126112
const results = await Promise.all(
127113
extensionsToInstall.map(async (ext) => {
128114
console.log(
129-
`Installing extension "${ext.slug}" on team "${account.name}" required by package(s): "${ext.packages.join(
115+
`Installing extension "${ext.slug}" on team "${accountId}" required by package(s): "${ext.packages.join(
130116
'",',
131117
)}"`,
132118
)

0 commit comments

Comments
 (0)