-
Notifications
You must be signed in to change notification settings - Fork 905
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
breaking: Remove cli-debugger-ui package #2583
breaking: Remove cli-debugger-ui package #2583
Conversation
a793852
to
1afce36
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.
Left some comments, overall looks good!
Very good spots @szymonrybczak — sorry I rebased my branch from my fork's |
For safety, latest push is a recreation of the commit after rebasing and re-deleting/searching for references. |
1afce36
to
5efa968
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.
Looks good now, thanks!
5efa968
to
60d96aa
Compare
@huntie there's a conflict in |
60d96aa
to
57ba392
Compare
@szymonrybczak Resolved. |
Summary:
Important
Breaking change: This is intended to target
main
/ the future18.x
release of CLI (aligned with React Native 0.79).Following our intent/announcement to remove Remote JS Debugging from React Native in 0.78, this PR removes the corresponding
cli-debugger-ui
frontend from the Community CLI.Also deletes related middleware/WebSocket support code in
cli-server-api
:/launch-js-devtools
debuggerProxyEndpoint
Test Plan:
✅ CI should pass
Checklist