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

feat: run-many prompts for projects #1064

Merged
merged 2 commits into from
Apr 23, 2021

Conversation

sandikbarr
Copy link
Contributor

@sandikbarr sandikbarr commented Apr 19, 2021

  • On the Nx: run-many command, when selecting projects it now prompts for selection from a quick pick list of projects with the given target instead of having the user type out the project entries.
  • The target flag has also been included in the userSetFlags in order to show the target with the command being built up.
    run-many projects quickpick from target

@nx-cloud
Copy link

nx-cloud bot commented Apr 19, 2021

Nx Cloud Report

CI ran the following commands for commit 28a47e4. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this branch

Status Command
#000000 nx run-many --all --parallel --target=build
#000000 nx run-many --all --parallel --target=lint
#000000 nx run-many --all --parallel --target=test

Sent with 💌 from NxCloud.

@sandikbarr sandikbarr requested a review from Cammisuli April 19, 2021 18:30
];
}

const flags = await selectFlags('run-many', options, 'nx', { target });
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Adding target to the userSetFlags parameter shows 'Execute: nx run-many --target=on the Command Palette instead of justnx: run-many` with the selected options. Seems to make more sense as that's how you'd use the run-many command.

@sandikbarr sandikbarr changed the title feat: run-many prompts from projects feat: run-many prompts for projects Apr 20, 2021
@Cammisuli Cammisuli merged commit 977bdf2 into nrwl:master Apr 23, 2021
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.

2 participants