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

Replace npm-run-all with npm-run-all2 #369

Merged
merged 1 commit into from
Apr 12, 2023
Merged

Replace npm-run-all with npm-run-all2 #369

merged 1 commit into from
Apr 12, 2023

Conversation

XhmikosR
Copy link
Member

@XhmikosR XhmikosR commented Apr 12, 2023

It's a maintained fork.

@julien-deramond are you on macOS? If so, can you try this branch locally?
@GeoSot IIRC you are on Linux. Could you also try this branch locally?

I plan to switch everywhere to npm-rul-all2 later if all goes well.

It's a maintained fork.
@XhmikosR XhmikosR marked this pull request as ready for review April 12, 2023 08:22
@GeoSot
Copy link
Member

GeoSot commented Apr 12, 2023

It seems to work as the previous version. I didn't notice any glitch on linux env

@julien-deramond
Copy link
Member

✅ Tested on macOS Ventura 13.3.1 with Fish and Bash, with no issues. Multiple scripts are well-executed.

@XhmikosR
Copy link
Member Author

Thanks, guys!

@XhmikosR XhmikosR merged commit cd45d51 into main Apr 12, 2023
@XhmikosR XhmikosR deleted the xmr/npm-run-all2 branch April 12, 2023 14:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants