fix: concurrent option is not working correctly

[jay-es]

Hello.

The value of --concurrent CLI option is passed as string like 'true' and '2'.
And tasks run serially when this option is set (even --concurrent true).

This is because the code to assign concurrency looks like this.

// node_modules/listr2/dist/listr.js:34-40 
this.concurrency = 1;
if (this.options.concurrent === true) {
    this.concurrency = Infinity;
}
else if (typeof this.options.concurrent === 'number') {
    this.concurrency = this.options.concurrent;
}

I have fixed this behavior.
Please review this PR, thanks.

before → after.

[iiroj]

@jay-es very nice catch! Thanks.

[codecov]

Codecov Report

Merging #950 (89739d9) into master (29a0b68) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master      #950   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           18        18           
  Lines          611       611           
  Branches       144       144           
=========================================
  Hits           611       611           

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 29a0b68...89739d9. Read the comment docs.

[iiroj]

@jay-es thanks again, merged!

[jay-es]

@iiroj Thank you!

[github-actions]

🎉 This PR is included in version 10.5.4 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀