Skip to content

Commit 67b6f71

Browse files
authored
fix: allow publishing to Snap Store to be disabled with snap-specific snap.publish options (#8896)
1 parent eb86464 commit 67b6f71

File tree

2 files changed

+11
-8
lines changed

2 files changed

+11
-8
lines changed

.changeset/three-doors-dream.md

+5
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
"app-builder-lib": patch
3+
---
4+
5+
fix: allow publishing to Snap Store to be disabled with snap specific publish options

packages/app-builder-lib/src/targets/snap.ts

+6-8
Original file line numberDiff line numberDiff line change
@@ -259,7 +259,7 @@ export default class SnapTarget extends Target {
259259
target: this,
260260
arch,
261261
packager,
262-
publishConfig: publishConfig == null ? { provider: "snapStore" } : publishConfig,
262+
publishConfig,
263263
})
264264
}
265265

@@ -269,16 +269,14 @@ export default class SnapTarget extends Target {
269269
}
270270

271271
function findSnapPublishConfig(config?: Configuration): SnapStoreOptions | null {
272+
const fallback: SnapStoreOptions = { provider: "snapStore" }
273+
272274
if (!config) {
273-
return null
275+
return fallback
274276
}
275277

276278
if (config.snap?.publish) {
277-
const configCandidate = findSnapPublishConfigInPublishNode(config.snap.publish)
278-
279-
if (configCandidate) {
280-
return configCandidate
281-
}
279+
return findSnapPublishConfigInPublishNode(config.snap.publish)
282280
}
283281

284282
if (config.linux?.publish) {
@@ -297,7 +295,7 @@ function findSnapPublishConfig(config?: Configuration): SnapStoreOptions | null
297295
}
298296
}
299297

300-
return null
298+
return fallback
301299
}
302300

303301
function findSnapPublishConfigInPublishNode(configPublishNode: Publish): SnapStoreOptions | null {

0 commit comments

Comments
 (0)