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

Improve lookupForRNModules by use __r.getModules if provided #468

Merged
merged 1 commit into from
Mar 14, 2020

Conversation

jhen0409
Copy link
Owner

It gets the actual module size and skips multiple __r(moduleId) attempts.

@jhen0409 jhen0409 merged commit 490cbaa into master Mar 14, 2020
@jhen0409 jhen0409 deleted the improve-lookup-rn-module branch March 14, 2020 02:04
jhen0409 added a commit that referenced this pull request Apr 4, 2020
* Upgrade react-devtools-core to v4 (#371)

* Use react-devtools-core v4 alpha

* Temporary made it work

* Update patch-modules

* Bump ws to v7

* Remove setDefaultThemeName usage

* Fix lint error

* Remove `How to use it with a real device?` tip

* Update docs/react-devtools-integration.md

* Remove DeltaBundler support

* Avoid box-sizing style for `.redux-container *`

* Create new auto_update.json feed for v0.11 usage

* Due to app breaking changes, we don't allow auto update for RNDebugger version < 0.11

* Update react-devtools-core to v4.3.0

* Fix E2E test

* Update doc

* v0.11.0-beta-0

* Update react-devtools-core to v4.4

* v0.11.0-beta-1

* v0.11.0-beta-2

* Improve lookupForRNModules by use __r.getModules if provided (#468)

* Update react-devtools-core to v4.5

* v0.11.0-beta-3

* Upgrade Electron to v8.0 (#456)

* Bump Electron version to v8.0.0-beta.3

* Bump to 8.0.0-beta.9

* Update electron and spectron version

* Update Electron to v8.0.1

* Update Electron to v8.0.2

* Update Electron to v8.1.1

* Bump node-abi version

* Update Electron to v8.2.0

* Update react-devtools-core to v4.6.0

* v0.11.0-beta-4

* E2E test: Update auto_update feed path

* Fix url of auto_update.json

* Update README.md

* Link v0.10 docs
cryptodev523 pushed a commit to cryptodev523/react-native-debugger that referenced this pull request Jan 27, 2022
cryptodev523 pushed a commit to cryptodev523/react-native-debugger that referenced this pull request Jan 27, 2022
* Upgrade react-devtools-core to v4 (jhen0409#371)

* Use react-devtools-core v4 alpha

* Temporary made it work

* Update patch-modules

* Bump ws to v7

* Remove setDefaultThemeName usage

* Fix lint error

* Remove `How to use it with a real device?` tip

* Update docs/react-devtools-integration.md

* Remove DeltaBundler support

* Avoid box-sizing style for `.redux-container *`

* Create new auto_update.json feed for v0.11 usage

* Due to app breaking changes, we don't allow auto update for RNDebugger version < 0.11

* Update react-devtools-core to v4.3.0

* Fix E2E test

* Update doc

* v0.11.0-beta-0

* Update react-devtools-core to v4.4

* v0.11.0-beta-1

* v0.11.0-beta-2

* Improve lookupForRNModules by use __r.getModules if provided (jhen0409#468)

* Update react-devtools-core to v4.5

* v0.11.0-beta-3

* Upgrade Electron to v8.0 (jhen0409#456)

* Bump Electron version to v8.0.0-beta.3

* Bump to 8.0.0-beta.9

* Update electron and spectron version

* Update Electron to v8.0.1

* Update Electron to v8.0.2

* Update Electron to v8.1.1

* Bump node-abi version

* Update Electron to v8.2.0

* Update react-devtools-core to v4.6.0

* v0.11.0-beta-4

* E2E test: Update auto_update feed path

* Fix url of auto_update.json

* Update README.md

* Link v0.10 docs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant