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(turborepo): new ui + watch mode #7962
Conversation
This stack of pull requests is managed by Graphite. Learn more about stacking. Join @NicholasLYang and the rest of your teammates on Graphite |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
5 Ignored Deployments
|
🟢 Turbopack Benchmark CI successful 🟢Thanks |
🟢 CI successful 🟢Thanks |
0d4f68d
to
02a5e85
Compare
09b44be
to
c090c69
Compare
02a5e85
to
24a0c13
Compare
c090c69
to
2050124
Compare
8931497
to
2152b42
Compare
2050124
to
c69ac3a
Compare
2152b42
to
84b3416
Compare
c69ac3a
to
e8ceb6f
Compare
84b3416
to
232f44d
Compare
e8ceb6f
to
53a852f
Compare
fe028d0
to
78937d6
Compare
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.
Overall this looks good, primarily concerned with testing the state machine that is our UI.
self.scroll.select(Some(selected_idx + 1)); | ||
} | ||
} | ||
} else { |
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.
So if a task gets restarted while it's still running it will error?
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.
That's currently not possible. We purposefully make it so the same task can't be run at the same time, and we haven't implemented cancelation logic. If a change event happens in the middle of a run, we save it and wait for the run to finish before starting up a new execution.
Co-authored-by: Chris Olszewski <[email protected]>
Description
Integrates new UI with watch mode. Pulls out the UI handle and sender outside the
Visitor
andRun
struct, so it can be owned by theWatchClient
. Also adds anEvent
for updating the task names, so on rediscovery we can keep using the same UI thread but just update the task names.Testing Instructions
Give it a shot!
Closes TURBO-2801