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: nx run in command palette and context menu #1065

Merged
merged 2 commits into from Apr 23, 2021

Conversation

sandikbarr
Copy link
Contributor

Adds support for nx run in the Command Palette
Prompts for a project then a target and supported flags (including configuration with a dropdown of the configuration options.)
run command

@nx-cloud
Copy link

nx-cloud bot commented Apr 20, 2021

Nx Cloud Report

CI ran the following commands for commit 3d0deac. 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.

@@ -49,6 +49,10 @@ export function registerCliTaskCommands(
);
});

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I am not really sure of the distinction between nx-task-commands and cli-task-commands. It feels like this cli-task-commands was originally angular CLI support vs Nx CLI. I started out putting the run command support in nx-task-commands, but the flow fit in a little better here with the other CLI commands and it has all the support for getting the executor/builder definition and prompting for flags.

Copy link
Member

Choose a reason for hiding this comment

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

yea, I noticed that as well. We'll eventually consolidate the two and just use Nx.

@sandikbarr sandikbarr changed the title feat: nx run in command palette feat: nx run in command palette and context menu Apr 23, 2021
@Cammisuli Cammisuli merged commit 9b5fd5c 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.

None yet

2 participants