-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
feat(core): added the ability to split command property into an array in nx:run-commands executor #20201
feat(core): added the ability to split command property into an array in nx:run-commands executor #20201
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
☁️ Nx Cloud ReportCI is running/has finished running commands for commit 4357a77. As they complete they will appear below. Click to see the status, the terminal output, and the build insights. 📂 See all runs for this CI Pipeline Execution
✅ Successfully ran 4 targets
Sent with 💌 from NxCloud. |
☁️ Nx Cloud ReportCI is running/has finished running commands for commit 5ef6b82. As they complete they will appear below. Click to see the status, the terminal output, and the build insights. 📂 See all runs for this CI Pipeline Execution ✅ Successfully ran 6 targets
Sent with 💌 from NxCloud. |
i am wondering whether this can be done something like:
|
this is ONE command, not several commands. like "\" in bash |
https://github.com/gearonix/sorry-nx I did this thing because I was tired of waiting. btw @FrozenPandaz, I already know this project quite well and am interested in working on nx. We can talk! |
6e103b7
to
5ef6b82
Compare
This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request. |
Current Behavior
In projects, you can often find teams that are too long. Example here:
Expected Behavior
Implemented splitting of the 'command' parameter, added the ability to pass it as an array to avoid such huge strings:
Added a couple of tests and changed schema.json to 'run-commands' executor.