Skip to content

Commit

Permalink
Fix publish behavior for CLI
Browse files Browse the repository at this point in the history
  • Loading branch information
garrettmflynn committed Dec 5, 2024
1 parent f4eae2e commit bca15ad
Show file tree
Hide file tree
Showing 4 changed files with 9 additions and 3 deletions.
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@
"@commoners/windows": "0.0.55",
"@vitest/coverage-v8": "^2.0.3",
"search-insights": "^2.15.0",
"commoners": "0.0.55",
"commoners": "0.0.56",
"vite": "^5.3.4",
"vitepress": "^1.3.1",
"vitest": "^2.0.3"
Expand Down
4 changes: 3 additions & 1 deletion packages/cli/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -113,9 +113,11 @@ cli.command('build [root]', 'Build the application in the specified directory',
.option('--config <path>', 'Specify a configuration file')
.action(async (root, options) => {

const { config: configPath, service, ...overrides } = options
const { config: configPath, service, sign, publish, ...overrides } = options
const { target } = overrides

overrides.build = { sign, publish }

await preprocessTarget(target)


Expand Down
2 changes: 1 addition & 1 deletion packages/cli/package.json
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
{
"name": "commoners",
"description": "Cross-Platform Development for the Rest of Us",
"version": "0.0.55",
"version": "0.0.56",
"type": "module",
"license": "MIT",
"engines": {
Expand Down
4 changes: 4 additions & 0 deletions tests/demo/commoners.config.ts
Original file line number Diff line number Diff line change
Expand Up @@ -53,6 +53,10 @@ const config = defineConfig({
// target: 'desktop' // Default target
// outDir: join(root, '_site'), // Custom output directory for all targets

// build: {
// sign: false, // Disable code signing
// },

pwa: {
manifest: { short_name }
},
Expand Down

0 comments on commit bca15ad

Please sign in to comment.