Skip to content

Commit 9f7aa65

Browse files
authored
fix(integ-runner): disable-update-workflow default is 'false' instead of false (#20073)
`disable-update-workflow` is a boolean, but the default value was a string 'false'. Update the default to be a boolean `false` value. ---- ### All Submissions: * [ ] Have you followed the guidelines in our [Contributing guide?](https://github.com/aws/aws-cdk/blob/master/CONTRIBUTING.md) ### Adding new Unconventional Dependencies: * [ ] This PR adds new unconventional dependencies following the process described [here](https://github.com/aws/aws-cdk/blob/master/CONTRIBUTING.md/#adding-new-unconventional-dependencies) ### New Features * [ ] Have you added the new feature to an [integration test](https://github.com/aws/aws-cdk/blob/master/INTEGRATION_TESTS.md)? * [ ] Did you use `yarn integ` to deploy the infrastructure and generate the snapshot (i.e. `yarn integ` without `--dry-run`)? *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
1 parent 721bd4b commit 9f7aa65

File tree

1 file changed

+2
-2
lines changed
  • packages/@aws-cdk/integ-runner/lib

1 file changed

+2
-2
lines changed

packages/@aws-cdk/integ-runner/lib/cli.ts

+2-2
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ async function main() {
2727
.options('max-workers', { type: 'number', desc: 'The max number of workerpool workers to use when running integration tests in parallel', default: 16 })
2828
.options('exclude', { type: 'boolean', desc: 'All tests should be run, except for the list of tests provided', default: false })
2929
.options('from-file', { type: 'string', desc: 'Import tests to include or exclude from a file' })
30-
.option('disable-update-workflow', { type: 'boolean', default: 'false', desc: 'If this is "true" then the stack update workflow will be disabled' })
30+
.option('disable-update-workflow', { type: 'boolean', default: false, desc: 'If this is "true" then the stack update workflow will be disabled' })
3131
.argv;
3232

3333
const pool = workerpool.pool(path.join(__dirname, '../lib/workers/extract/index.js'), {
@@ -92,7 +92,7 @@ async function main() {
9292
clean: argv.clean,
9393
dryRun: argv['dry-run'],
9494
verbose: argv.verbose,
95-
updateWorkflow: !!argv['disable-update-workflow'],
95+
updateWorkflow: !argv['disable-update-workflow'],
9696
});
9797

9898
if (argv.clean === false) {

0 commit comments

Comments
 (0)