search

open-cli-tools / concurrently

Run commands concurrently. Like `npm run watch-js & npm run watch-less` but better.
https://www.npmjs.com/package/concurrently
MIT License
6.98k stars 227 forks source link

Add a state property to Command #455

Closed gustavohenke closed 8 months ago

gustavohenke commented 8 months ago

Title. The intention is that this will make it easier to tell exactly where the command is. My idea is that it can be used for #433, since it might be necessary to make a distinction between a command that was never started, and everything else.

Also marked it a breaking change because I removed the #exited property, as #state supersedes it. Booleans do a poor job sometimes :)

Most LOC in this PR are coming from moving a few tests around.

coveralls commented 8 months ago

Coverage Status

coverage: 99.2% (+0.003%) from 99.197% when pulling 212d57949c7d110a4d7175639e2c6a81cebf9773 on command-state into 0c64306f2bd7bf36ef5177afe092bf2875b0bc01 on main.

gustavohenke commented 8 months ago

happy new year 🥳