Skip to content

Commit caa440e

Browse files
committed
fix(setup): ensure we only skip tasks that should be skipped
no issue - fix ssl not getting run at all even if it should be
1 parent 3e54db0 commit caa440e

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

lib/commands/setup.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -115,7 +115,7 @@ class SetupCommand extends Command {
115115
// TODO: this depends on Listr private API, probably should find a better way
116116
let skipped = stage.dependencies.filter(dep => !taskMap[dep] || taskMap[dep]._task.isSkipped());
117117

118-
if (skipped) {
118+
if (skipped && skipped.length) {
119119
let plural = skipped.length > 1;
120120
this.ui.log(`Task ${stage.name} depends on the '${skipped.join('\', \'')}' ${plural ? 'stages' : 'stage'}, which ${plural ? 'were' : 'was'} skipped.`, 'gray');
121121
return task.skip();

0 commit comments

Comments
 (0)