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

RN: Support Private Methods & Properties #39318

Closed
wants to merge 1 commit into from

Conversation

yungsters
Copy link
Contributor

Summary:
Updates react-native/babel-preset to support private methods and properties.

Changelog:
[General][Added] Added plugins for private methods and properties to react-native/babel-preset.

Differential Revision: D49029339

@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. p: Facebook Partner: Facebook Partner labels Sep 6, 2023
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D49029339

@analysis-bot
Copy link

analysis-bot commented Sep 6, 2023

Platform Engine Arch Size (bytes) Diff
android hermes arm64-v8a 8,966,259 -1
android hermes armeabi-v7a n/a --
android hermes x86 n/a --
android hermes x86_64 n/a --
android jsc arm64-v8a 9,558,821 +49
android jsc armeabi-v7a n/a --
android jsc x86 n/a --
android jsc x86_64 n/a --

Base commit: 48c297e
Branch: main

@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D49029339

yungsters added a commit to yungsters/react-native that referenced this pull request Sep 8, 2023
Summary:
Pull Request resolved: facebook#39318

Updates `react-native/babel-preset` to support private methods and properties.

Changelog:
[General][Added] Added plugins for private methods and properties to `react-native/babel-preset`.

Reviewed By: asiandrummer

Differential Revision: D49029339

fbshipit-source-id: e3194ebf2e058acb77662e83f943fbd5dbe4ec77
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D49029339

yungsters added a commit to yungsters/react-native that referenced this pull request Sep 11, 2023
Summary:
Pull Request resolved: facebook#39318

Updates `react-native/babel-preset` to support private methods and properties.

Changelog:
[General][Added] Added plugins for private methods and properties to `react-native/babel-preset`.

Reviewed By: asiandrummer

Differential Revision: D49029339

fbshipit-source-id: 5982605383702979b4598323c4bde9a9e8b12ab3
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D49029339

yungsters added a commit to yungsters/react-native that referenced this pull request Sep 11, 2023
Summary:
Pull Request resolved: facebook#39318

Updates `react-native/babel-preset` to support private methods and properties.

Changelog:
[General][Added] Added plugins for private methods and properties to `react-native/babel-preset`.

Reviewed By: asiandrummer

Differential Revision: D49029339

fbshipit-source-id: 09067c741ba2007127106fa4bb29fcd175c36c4b
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D49029339

yungsters added a commit to yungsters/react-native that referenced this pull request Sep 11, 2023
Summary:
Pull Request resolved: facebook#39318

Updates `react-native/babel-preset` to support private methods and properties.

Changelog:
[General][Added] Added plugins for private methods and properties to `react-native/babel-preset`.

Reviewed By: asiandrummer

Differential Revision: D49029339

fbshipit-source-id: e3a06bd8d2f5ae74aca749acaa5c9b8fd1839421
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D49029339

@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D49029339

yungsters added a commit to yungsters/react-native that referenced this pull request Sep 11, 2023
Summary:
Pull Request resolved: facebook#39318

Updates `react-native/babel-preset` to support private methods and properties.

Changelog:
[General][Added] Added plugins for private methods and properties to `react-native/babel-preset`.

Reviewed By: asiandrummer

Differential Revision: D49029339

fbshipit-source-id: 3b6595977b8a761780cea7558c23cb3eb07ad05f
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D49029339

yungsters added a commit to yungsters/react-native that referenced this pull request Sep 11, 2023
Summary:
Pull Request resolved: facebook#39318

Updates `react-native/babel-preset` to support private methods and properties.

Changelog:
[General][Added] Added plugins for private methods and properties to `react-native/babel-preset`.

Reviewed By: asiandrummer

Differential Revision: D49029339

fbshipit-source-id: b1295ae93ffe88336ba500774f76dbb8ddd43aeb
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D49029339

yungsters added a commit to yungsters/react-native that referenced this pull request Sep 11, 2023
Summary:
Pull Request resolved: facebook#39318

Updates `react-native/babel-preset` to support private methods and properties.

Changelog:
[General][Added] Added plugins for private methods and properties to `react-native/babel-preset`.

Reviewed By: asiandrummer

Differential Revision: D49029339

fbshipit-source-id: 1092930d931a2b59ba66b43b6be2e9b1863f14c0
Summary:
Pull Request resolved: facebook#39318

Updates `react-native/babel-preset` to support private methods and properties.

Changelog:
[General][Added] Added plugins for private methods and properties to `react-native/babel-preset`.

Reviewed By: asiandrummer

Differential Revision: D49029339

fbshipit-source-id: 2df73ad9ee28471ec5cc1c1778a70de060c6945a
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D49029339

@github-actions
Copy link

github-actions bot commented Sep 11, 2023

Warnings
⚠️ One hour and a half have passed and the E2E jobs haven't finished yet.

Generated by 🚫 dangerJS against a331caa

@facebook-github-bot facebook-github-bot added the Merged This PR has been merged. label Sep 11, 2023
@facebook-github-bot
Copy link
Contributor

This pull request has been merged in db4a253.

yungsters added a commit to yungsters/react-native that referenced this pull request Sep 11, 2023
Summary:
Switches `EventEmitter` to use a private property.

Support for private properties was [only recently added](facebook#39318), so this will be the first end-to-end validation of support in the `facebook/react-native` project.

Changelog:
[Internal]

Differential Revision: D49167908
yungsters added a commit to yungsters/react-native that referenced this pull request Sep 12, 2023
Summary:
Pull Request resolved: facebook#39395

Switches `EventEmitter` to use a private property.

Support for private properties was [only recently added](facebook#39318), so this will be the first end-to-end validation of support in the `facebook/react-native` project.

Changelog:
[Internal]

Differential Revision: D49167908

fbshipit-source-id: f2e9f9d6e3b73639b92ad6fe9e652aaad20dd3df
yungsters added a commit to yungsters/react-native that referenced this pull request Sep 12, 2023
Summary:

Switches `EventEmitter` to use a private property.

Support for private properties was [only recently added](facebook#39318), so this will be the first end-to-end validation of support in the `facebook/react-native` project.

Changelog:
[Internal]

Reviewed By: voideanvalue

Differential Revision: D49167908
facebook-github-bot pushed a commit that referenced this pull request Sep 12, 2023
Summary:
Pull Request resolved: #39395

Switches `EventEmitter` to use a private property.

Support for private properties was [only recently added](#39318), so this will be the first end-to-end validation of support in the `facebook/react-native` project.

Changelog:
[Internal]

Reviewed By: voideanvalue

Differential Revision: D49167908

fbshipit-source-id: 6a496ed130bfe5671663166626b1acbd80b0e9c3
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported Merged This PR has been merged. p: Facebook Partner: Facebook Partner
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants