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: Use a Private Property in EventEmitter #39395

Closed
wants to merge 1 commit into from

Conversation

yungsters
Copy link
Contributor

Summary:
Switches EventEmitter to use a private property.

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

Changelog:
[Internal]

Differential Revision: D49167908

@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 11, 2023
@facebook-github-bot
Copy link
Contributor

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

@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 e8fc708

@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. 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
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
Copy link
Contributor

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

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

This pull request has been merged in 06b5c6e.

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.

2 participants