-
Notifications
You must be signed in to change notification settings - Fork 2.5k
feat(core): add nx console messaging to TUI #31148
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
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Skipped Deployment
|
View your CI Pipeline Execution ↗ for commit 587ccb5.
☁️ Nx Cloud last updated this comment at |
Failed to publish a PR release of this pull request, triggered by @Cammisuli. |
6820ac2
to
f475a11
Compare
Failed to publish a PR release of this pull request, triggered by @Cammisuli. |
4c3bd8a
to
572053e
Compare
Failed to publish a PR release of this pull request, triggered by @Cammisuli. |
Failed to publish a PR release of this pull request, triggered by @Cammisuli. |
817d52f
to
05f858f
Compare
9525149
to
aacc95a
Compare
🐳 We have a release for that!This PR has a release associated with it. You can try it out using this command: npx create-nx-workspace@0.0.0-pr-31148-817d52f my-workspace Or just copy this version and use it in your own command: 0.0.0-pr-31148-817d52f
To request a new release for this pull request, mention someone from the Nx team or the |
0dc8a95
to
01708fd
Compare
aacc95a
to
e5c0eb5
Compare
8261e9b
to
d4a1827
Compare
d4a1827
to
d063c0f
Compare
01708fd
to
7b778de
Compare
🐳 We have a release for that!This PR has a release associated with it. You can try it out using this command: npx create-nx-workspace@0.0.0-pr-31148-d063c0f my-workspace Or just copy this version and use it in your own command: 0.0.0-pr-31148-d063c0f
To request a new release for this pull request, mention someone from the Nx team or the |
🐳 We have a release for that!This PR has a release associated with it. You can try it out using this command: npx create-nx-workspace@0.0.0-pr-31148-de3e81b my-workspace Or just copy this version and use it in your own command: 0.0.0-pr-31148-de3e81b
To request a new release for this pull request, mention someone from the Nx team or the |
7b778de
to
91afe69
Compare
de3e81b
to
77df785
Compare
77df785
to
587ccb5
Compare
🐳 We have a release for that!This PR has a release associated with it. You can try it out using this command: npx create-nx-workspace@0.0.0-pr-31148-587ccb5 my-workspace Or just copy this version and use it in your own command: 0.0.0-pr-31148-587ccb5
To request a new release for this pull request, mention someone from the Nx team or the |
.and_then(|c| c.start_running_tasks()); | ||
} | ||
Action::Tick => { | ||
self.console_messenger.as_ref().and_then(|messenger| { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we throttle/debounce in some way? Updating everything on every tick is quite intense, no?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please can we add some more logs in the happy path when messages are being send to console?
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
There is no communication between Nx CLI and Nx Console
Expected Behavior
This enables a connection between the Nx TUI app and Nx Console so that we can send messages to console. This is used to update MCP tools on Nx Console to assist with LLMs
Related Issue(s)
Fixes #