-
Notifications
You must be signed in to change notification settings - Fork 24k
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
Align with code-gen output Sample Turbo Modules #44640
Conversation
This pull request was exported from Phabricator. Differential Revision: D57624824 |
Summary: These modules are manually typed and don't use `react-native-codgen` (as a show case to demonstrates what code-gen produces). There are some typos in the manually written code and they don't exactly match the JS Spec used at runtime: https://github.com/facebook/react-native/blob/main/packages/react-native/src/private/specs/modules/NativeSampleTurboModule.js#L45-L48 Changelog: [Internal][Fixed] Align with code-gen output Sample Turbo Modules Differential Revision: D57624824
95968f9
to
c5e0e10
Compare
This pull request was exported from Phabricator. Differential Revision: D57624824 |
Summary: These modules are manually typed and don't use `react-native-codgen` (as a show case to demonstrates what code-gen produces). There are some typos in the manually written code and they don't exactly match the JS Spec used at runtime: https://github.com/facebook/react-native/blob/main/packages/react-native/src/private/specs/modules/NativeSampleTurboModule.js#L45-L48 Changelog: [Internal][Fixed] Align with code-gen output Sample Turbo Modules Differential Revision: D57624824
c5e0e10
to
5d9dffc
Compare
This pull request was exported from Phabricator. Differential Revision: D57624824 |
Base commit: cd1ddbb |
Summary: These modules are manually typed and don't use `react-native-codgen` (as a show case to demonstrates what code-gen produces). There are some typos in the manually written code and they don't exactly match the JS Spec used at runtime: https://github.com/facebook/react-native/blob/main/packages/react-native/src/private/specs/modules/NativeSampleTurboModule.js#L45-L48 Changelog: [Internal][Fixed] Align with code-gen output Sample Turbo Modules Differential Revision: D57624824
Summary: These modules are manually typed and don't use `react-native-codgen` (as a show case to demonstrates what code-gen produces). There are some typos in the manually written code and they don't exactly match the JS Spec used at runtime: https://github.com/facebook/react-native/blob/main/packages/react-native/src/private/specs/modules/NativeSampleTurboModule.js#L45-L48 Changelog: [Internal][Fixed] Align with code-gen output Sample Turbo Modules Differential Revision: D57624824
…4640) Summary: These modules are manually typed and don't use `react-native-codgen` (as a show case to demonstrates what code-gen produces). There are some typos in the manually written code and they don't exactly match the JS Spec used at runtime: https://github.com/facebook/react-native/blob/main/packages/react-native/src/private/specs/modules/NativeSampleTurboModule.js#L45-L48 Changelog: [Internal][Fixed] Align with code-gen output Sample Turbo Modules Differential Revision: D57624824
This pull request has been merged in 24aed6e. |
This pull request was successfully merged by @christophpurrer in 24aed6e. When will my fix make it into a release? | How to file a pick request? |
Summary: Pull Request resolved: facebook#44640 These modules are manually typed and don't use `react-native-codgen` (as a show case to demonstrates what code-gen produces). There are some typos in the manually written code and they don't exactly match the JS Spec used at runtime: https://github.com/facebook/react-native/blob/main/packages/react-native/src/private/specs/modules/NativeSampleTurboModule.js#L45-L48 Changelog: [Internal][Fixed] Align with code-gen output Sample Turbo Modules Reviewed By: rshest Differential Revision: D57624824 fbshipit-source-id: 023f2318630aa342758d5ea495e6298cb8884ae1
Summary:
These modules are manually typed and don't use
react-native-codgen
(as a show case to demonstrates what code-gen produces).There are some typos in the manually written code and they don't exactly match the JS Spec used at runtime:
https://github.com/facebook/react-native/blob/main/packages/react-native/src/private/specs/modules/NativeSampleTurboModule.js#L45-L48
Changelog: [Internal][Fixed] Align with code-gen output Sample Turbo Modules
Differential Revision: D57624824