Skip to content

Commit

Permalink
feat: Build for production by default when making/packaging
Browse files Browse the repository at this point in the history
`ember electron:make` and `ember electron:package` are much more similar to `ember electron:deploy` (which defaults to building for production) than they are to `ember electron:build` (which defaults to building for development). So this switches the default environment for those commands to `production`.

Making or packaging is much more similar to
  • Loading branch information
bendemboski committed Nov 21, 2019
1 parent ea7ff75 commit 5484f6f
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion lib/commands/make.js
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ module.exports = BaseCommand.extend({
description: 'Packages your Ember + Electron app into shippable installers',

availableOptions: [
{ name: 'environment', type: String, default: 'development', aliases: ['e', { 'dev': 'development' }, { 'prod': 'production' }], description: 'Possible values are "development", "production", and "test".' },
{ name: 'environment', type: String, default: 'production', aliases: ['e', { 'dev': 'development' }, { 'prod': 'production' }], description: 'Possible values are "development", "production", and "test".' },
{ name: 'platform', type: String, aliases: ['p'] },
{ name: 'arch', type: String, aliases: ['a'] },
{ name: 'targets', type: String, aliases: ['t'], description: 'Override the build targets specified in your electron-forge config.' },
Expand Down
2 changes: 1 addition & 1 deletion lib/commands/package.js
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ module.exports = BaseCommand.extend({
description: 'Builds & packages your Electron app',

availableOptions: [
{ name: 'environment', type: String, default: 'development', aliases: ['e', { 'dev': 'development' }, { 'prod': 'production' }], description: 'Possible values are "development", "production", and "test".' },
{ name: 'environment', type: String, default: 'production', aliases: ['e', { 'dev': 'development' }, { 'prod': 'production' }], description: 'Possible values are "development", "production", and "test".' },
{ name: 'platform', type: String, aliases: ['p'] },
{ name: 'arch', type: String, aliases: ['a'] },
{ name: 'skip-build', type: Boolean, aliases: ['s'], description: 'Skip the Ember build step and use an already-build from the output path.' },
Expand Down

0 comments on commit 5484f6f

Please sign in to comment.