Skip to content
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

Use maintained fork of npm-run-all #20523

Merged

Conversation

voxpelli
Copy link
Contributor

Similar to #20522 it can be hard for users of npm-run-all to find out that there's a maintained fork: npm-run-all2

In my drive to raise awareness for eslint-plugin-n I'm therefore also making PR:s for npm-run-all2 for projects that contain npm-run-all.

@NullVoxPopuli
Copy link
Contributor

Hello! our ecosystem has actually migrated to concurrently -- any chance you have the energy to make that swap? thank you!!

@voxpelli
Copy link
Contributor Author

I myself moved the other way, from concurrently to npm-run-all so I'm not the right person to suggest how to add that.

I would suggest merging this in the meanwhile to get the latest and greatest dependencies.

@kategengler
Copy link
Member

@bertdeblock Do you have thoughts on this?

@bertdeblock
Copy link
Member

I'd say if npm-run-all is dead, merging this seems fine. A switch to concurrently can still be done afterwards if desired?

Similar to emberjs#20522 it can be hard for users of `npm-run-all` to find out that there's a maintained fork: https://github.com/bcomnes/npm-run-all2

In my drive to raise awareness for `eslint-plugin-n` I'm therefore also making PR:s for `npm-run-all2` for projects that contain `npm-run-all`
@kategengler kategengler force-pushed the voxpelli/switch-to-maintained-npm-run-all branch from dccae83 to b836016 Compare December 12, 2023 16:28
@kategengler kategengler merged commit dd85fe6 into emberjs:main Dec 12, 2023
19 checks passed
@voxpelli voxpelli deleted the voxpelli/switch-to-maintained-npm-run-all branch December 12, 2023 16:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants