Skip to content

Conversation

byCedric
Copy link
Member

@byCedric byCedric commented Dec 12, 2023

Why

Stacked on top of #25881
Correction of #25671

This was in place for the dev menu, and I deleted it a bit too early in #25671.

We can only delete this once we fully swapped over to @react-native/dev-middleware, which is currently only used for the inspector proxy infrastructure.

How

Test Plan

  • $ bun create expo ./test-devmenu-debugger --template blank@50
  • $ cd ./test-devmenu-debugger
  • $ bun expo install expo-dev-client
  • $ bun expo start
  • Press m in terminal
  • Press "Inspect JS" in the menu within the app

Checklist

@byCedric byCedric requested a review from EvanBacon as a code owner December 12, 2023 19:02
@expo-bot
Copy link
Collaborator

Hi there! 👋 I'm a bot whose goal is to ensure your contributions meet our guidelines.

I've found some issues in your pull request that should be addressed (click on them for more details) 👇

⚠️ Suggestion: Missing changelog entries


Your changes should be noted in the changelog. Read Updating Changelogs guide and consider adding an appropriate entry to the following changelogs:


Generated by ExpoBot 🤖 against 85fda55

@expo-bot expo-bot added the bot: suggestions ExpoBot has some suggestions label Dec 12, 2023
@byCedric byCedric merged commit eff591c into @gabrieldonadel/fix-inspector Dec 12, 2023
@byCedric byCedric deleted the @bycedric/cli/add-back-legacy-inspector-handler branch December 12, 2023 19:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants