-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Deps(repo): Switch from npm-run-all
to npm-run-all2
fork
#1337
Conversation
✅ Deploy Preview for spirit-design-system-validations canceled.
|
✅ Deploy Preview for spirit-design-system-storybook ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for spirit-design-system ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Do we plan to keep this even if it is only a maintenance fork? I understand that it is better then the non-maintained one, but I would like to know if we have a plan to move somewhere else :) |
In this case "maintenance" means not only updates but also some development for future node compatibility and some babel issues, etc. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🙈
Anyway, do we use the --parallel
flag anywhere? Because if not, I don't see a reason for using a dependency instead of writing something as simple as yarn task1 && yarn task2
…
web: Anyway, this is a dev dependency and it is a part of the DX. I do not want to write |
3f1f6e5
to
b4fe645
Compare
* the original library is unmaintained for more than 5 years * the `npm-run-all2` is a maintanance fork * @see: https://github.com/bcomnes/npm-run-all2
b4fe645
to
5209aa5
Compare
Description
npm-run-all2
is a maintenance forkAdditional context
https://github.com/bcomnes/npm-run-all2
Issue reference