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

[pull] master from facebook:master #346

Merged
merged 3 commits into from
Oct 29, 2020
Merged

Conversation

pull[bot]
Copy link

@pull pull bot commented Oct 29, 2020

See Commits and Changes for more details.


Created by pull[bot]

Can you help keep this open source service alive? 💖 Please sponsor : )

safaiyeh and others added 3 commits October 29, 2020 07:40
Summary:
Update engines to node >= 12

## Changelog

<!-- Help reviewers and the release process by writing your own changelog entry. For an example, see:
https://github.com/facebook/react-native/wiki/Changelog
-->

[General] [Changed] - Update node engine to version 12

Pull Request resolved: #30252

Reviewed By: yungsters

Differential Revision: D24558093

Pulled By: fkgozali

fbshipit-source-id: 7a10bb935f89ba3374b0909161db9974e1fc477a
Summary:
Changelog:
Fix the cloneWithChildren implementation that was not copying the list of children on the java object.

We were missing on copying the list of children when cloning. This is pretty bad as it means that the clone operation was mutating the old node as well as the new. When multiple threads were involved this could cause crashes.

Reviewed By: SidharthGuglani

Differential Revision: D24565307

fbshipit-source-id: 4e2e111db389e25c315ce7603b4018ac695bb0f1
Summary:
The current available version of eslint-plugin-react is 7.21.4.
Update it to 7.21.5 to be able to use function-component-definition rule https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/function-component-definition.md

Changelog: [Internal]

Reviewed By: cpojer

Differential Revision: D24619420

fbshipit-source-id: 3097c4d920aa41b2dcced8d0ef5224b04a36effe
@pull pull bot added the ⤵️ pull label Oct 29, 2020
@pull pull bot merged commit 1b71ec4 into MLH-Fellowship:master Oct 29, 2020
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.

2 participants