command-${string}-formatted --success condition fails with dashes

[eventualbuddha]

This should use the exit code from the first command (exit 1), but doesn't because the successCondition splits on -:

concurrently \
  --names 'this-one,that-one' \
  --success command-this-one \
  'exit 1' \
  'exit 0'

This version does work:

concurrently \
  --names 'this,that' \
  --success command-that \
  'exit 1' \
  'exit 0'

[gustavohenke]

Thanks for spotting this 🤦

[eventualbuddha]

Related, passing a command name that doesn't exist just silently ignores it. Maybe also true with indexes?

[paescuj]

Related, passing a command name that doesn't exist just silently ignores it. Maybe also true with indexes?

Thanks for your input! We've discussed this in #318 (comment).

[gustavohenke]

Fix out in v7.2.1.