Skip to content

Commit

Permalink
fix: use the skip default value for skip cli arg (#211)
Browse files Browse the repository at this point in the history
Even though the defaults for both `skip` and `scripts` were empty
objects, the code was referencing the default value for `scripts` in
both places, which could lead to a bug in the future if the default for
either changed at some point in the future.
  • Loading branch information
spikegrobstein authored and nexdrew committed Nov 21, 2017
1 parent 709dae1 commit 3fdd7fa
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion command.js
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,7 @@ module.exports = require('yargs')
})
.option('skip', {
describe: 'Map of steps in the release process that should be skipped',
default: defaults.scripts
default: defaults.skip
})
.option('dry-run', {
type: 'boolean',
Expand Down

0 comments on commit 3fdd7fa

Please sign in to comment.