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

Remove YGNodeMarkDirtyAndPropagateToDescendants #1448

Closed
wants to merge 1 commit into from

Conversation

NickGerleman
Copy link
Contributor

Summary: This should not be part of Yoga's API. If benchmarks want to do this, they still can (though I don't know the ones we have for it are super valuable).

Differential Revision: D50963933

@facebook-github-bot
Copy link
Contributor

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

NickGerleman added a commit to NickGerleman/yoga that referenced this pull request Nov 3, 2023
Summary:

This should not be part of Yoga's API. If benchmarks want to do this, they still can (though I don't know the ones we have for it are super valuable).

Reviewed By: javache

Differential Revision: D50963933
@facebook-github-bot
Copy link
Contributor

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

NickGerleman added a commit to NickGerleman/react-native that referenced this pull request Nov 3, 2023
Summary:
X-link: facebook/yoga#1448

This should not be part of Yoga's API. If benchmarks want to do this, they still can (though I don't know the ones we have for it are super valuable).

Reviewed By: javache

Differential Revision: D50963933
NickGerleman added a commit to NickGerleman/yoga that referenced this pull request Nov 3, 2023
Summary:
X-link: facebook/react-native#41305


This should not be part of Yoga's API. If benchmarks want to do this, they still can (though I don't know the ones we have for it are super valuable).

Reviewed By: javache

Differential Revision: D50963933
NickGerleman added a commit to NickGerleman/react-native that referenced this pull request Nov 3, 2023
Summary:

X-link: facebook/yoga#1448

This should not be part of Yoga's API. If benchmarks want to do this, they still can (though I don't know the ones we have for it are super valuable).

Reviewed By: javache

Differential Revision: D50963933
NickGerleman added a commit to NickGerleman/react-native that referenced this pull request Nov 3, 2023
Summary:
Pull Request resolved: facebook#41305

X-link: facebook/yoga#1448

This should not be part of Yoga's API. If benchmarks want to do this, they still can (though I don't know the ones we have for it are super valuable).

Reviewed By: javache

Differential Revision: D50963933

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

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

NickGerleman added a commit to NickGerleman/yoga that referenced this pull request Nov 3, 2023
Summary:
X-link: facebook/react-native#41305

Pull Request resolved: facebook#1448

This should not be part of Yoga's API. If benchmarks want to do this, they still can (though I don't know the ones we have for it are super valuable).

Reviewed By: javache

Differential Revision: D50963933

fbshipit-source-id: d788748d98248cde724d229a7730757879e9e306
NickGerleman added a commit to NickGerleman/yoga that referenced this pull request Nov 3, 2023
Summary:
X-link: facebook/react-native#41305


This should not be part of Yoga's API. If benchmarks want to do this, they still can (though I don't know the ones we have for it are super valuable).

Reviewed By: javache

Differential Revision: D50963933
NickGerleman added a commit to NickGerleman/react-native that referenced this pull request Nov 3, 2023
Summary:

X-link: facebook/yoga#1448

This should not be part of Yoga's API. If benchmarks want to do this, they still can (though I don't know the ones we have for it are super valuable).

Reviewed By: javache

Differential Revision: D50963933
@facebook-github-bot
Copy link
Contributor

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

NickGerleman added a commit to NickGerleman/yoga that referenced this pull request Nov 3, 2023
Summary:
X-link: facebook/react-native#41305


This should not be part of Yoga's API. If benchmarks want to do this, they still can (though I don't know the ones we have for it are super valuable).

Reviewed By: javache

Differential Revision: D50963933
NickGerleman added a commit to NickGerleman/react-native that referenced this pull request Nov 3, 2023
Summary:

X-link: facebook/yoga#1448

This should not be part of Yoga's API. If benchmarks want to do this, they still can (though I don't know the ones we have for it are super valuable).

Reviewed By: javache

Differential Revision: D50963933
NickGerleman added a commit to NickGerleman/yoga that referenced this pull request Nov 3, 2023
Summary:
X-link: facebook/react-native#41305


This should not be part of Yoga's API. If benchmarks want to do this, they still can (though I don't know the ones we have for it are super valuable).

Reviewed By: javache

Differential Revision: D50963933
NickGerleman added a commit to NickGerleman/react-native that referenced this pull request Nov 3, 2023
Summary:

X-link: facebook/yoga#1448

This should not be part of Yoga's API. If benchmarks want to do this, they still can (though I don't know the ones we have for it are super valuable).

Reviewed By: javache

Differential Revision: D50963933
NickGerleman added a commit to NickGerleman/yoga that referenced this pull request Nov 4, 2023
Summary:
X-link: facebook/react-native#41305


This should not be part of Yoga's API. If benchmarks want to do this, they still can (though I don't know the ones we have for it are super valuable).

Reviewed By: javache

Differential Revision: D50963933
NickGerleman added a commit to NickGerleman/react-native that referenced this pull request Nov 4, 2023
Summary:

X-link: facebook/yoga#1448

This should not be part of Yoga's API. If benchmarks want to do this, they still can (though I don't know the ones we have for it are super valuable).

Reviewed By: javache

Differential Revision: D50963933
NickGerleman added a commit to NickGerleman/yoga that referenced this pull request Nov 6, 2023
Summary:
X-link: facebook/react-native#41305


This should not be part of Yoga's API. If benchmarks want to do this, they still can (though I don't know the ones we have for it are super valuable).

Reviewed By: javache

Differential Revision: D50963933
NickGerleman added a commit to NickGerleman/react-native that referenced this pull request Nov 6, 2023
Summary:

X-link: facebook/yoga#1448

This should not be part of Yoga's API. If benchmarks want to do this, they still can (though I don't know the ones we have for it are super valuable).

Reviewed By: javache

Differential Revision: D50963933
Summary:
X-link: facebook/react-native#41305


This should not be part of Yoga's API. If benchmarks want to do this, they still can (though I don't know the ones we have for it are super valuable).

Reviewed By: javache

Differential Revision: D50963933
NickGerleman added a commit to NickGerleman/react-native that referenced this pull request Nov 6, 2023
Summary:

X-link: facebook/yoga#1448

This should not be part of Yoga's API. If benchmarks want to do this, they still can (though I don't know the ones we have for it are super valuable).

Reviewed By: javache

Differential Revision: D50963933
@facebook-github-bot
Copy link
Contributor

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

NickGerleman added a commit to NickGerleman/yoga that referenced this pull request Nov 6, 2023
Summary:
X-link: facebook/react-native#41305


This should not be part of Yoga's API. If benchmarks want to do this, they still can (though I don't know the ones we have for it are super valuable).

Reviewed By: javache

Differential Revision: D50963933
NickGerleman added a commit to NickGerleman/react-native that referenced this pull request Nov 7, 2023
Summary:

X-link: facebook/yoga#1448

This should not be part of Yoga's API. If benchmarks want to do this, they still can (though I don't know the ones we have for it are super valuable).

Reviewed By: javache

Differential Revision: D50963933
NickGerleman added a commit to NickGerleman/yoga that referenced this pull request Nov 7, 2023
Summary:
X-link: facebook/react-native#41305


This should not be part of Yoga's API. If benchmarks want to do this, they still can (though I don't know the ones we have for it are super valuable).

Reviewed By: javache

Differential Revision: D50963933
NickGerleman added a commit to NickGerleman/yoga that referenced this pull request Nov 7, 2023
Summary:
X-link: facebook/react-native#41305


This should not be part of Yoga's API. If benchmarks want to do this, they still can (though I don't know the ones we have for it are super valuable).

Reviewed By: javache

Differential Revision: D50963933
NickGerleman added a commit to NickGerleman/react-native that referenced this pull request Nov 7, 2023
Summary:

X-link: facebook/yoga#1448

This should not be part of Yoga's API. If benchmarks want to do this, they still can (though I don't know the ones we have for it are super valuable).

Reviewed By: javache

Differential Revision: D50963933
NickGerleman added a commit to NickGerleman/react-native that referenced this pull request Nov 7, 2023
Summary:

X-link: facebook/yoga#1448

This should not be part of Yoga's API. If benchmarks want to do this, they still can (though I don't know the ones we have for it are super valuable).

Reviewed By: javache

Differential Revision: D50963933
NickGerleman added a commit to NickGerleman/yoga that referenced this pull request Nov 7, 2023
Summary:
X-link: facebook/react-native#41305


This should not be part of Yoga's API. If benchmarks want to do this, they still can (though I don't know the ones we have for it are super valuable).

Reviewed By: javache

Differential Revision: D50963933
NickGerleman added a commit to NickGerleman/react-native that referenced this pull request Nov 7, 2023
Summary:

X-link: facebook/yoga#1448

This should not be part of Yoga's API. If benchmarks want to do this, they still can (though I don't know the ones we have for it are super valuable).

Reviewed By: javache

Differential Revision: D50963933
NickGerleman added a commit to NickGerleman/yoga that referenced this pull request Nov 7, 2023
Summary:
X-link: facebook/react-native#41305


This should not be part of Yoga's API. If benchmarks want to do this, they still can (though I don't know the ones we have for it are super valuable).

Reviewed By: javache

Differential Revision: D50963933
NickGerleman added a commit to NickGerleman/react-native that referenced this pull request Nov 7, 2023
Summary:

X-link: facebook/yoga#1448

This should not be part of Yoga's API. If benchmarks want to do this, they still can (though I don't know the ones we have for it are super valuable).

Reviewed By: javache

Differential Revision: D50963933
NickGerleman added a commit to NickGerleman/yoga that referenced this pull request Nov 7, 2023
Summary:
X-link: facebook/react-native#41305


This should not be part of Yoga's API. If benchmarks want to do this, they still can (though I don't know the ones we have for it are super valuable).

Reviewed By: javache

Differential Revision: D50963933
NickGerleman added a commit to NickGerleman/react-native that referenced this pull request Nov 7, 2023
Summary:

X-link: facebook/yoga#1448

This should not be part of Yoga's API. If benchmarks want to do this, they still can (though I don't know the ones we have for it are super valuable).

Reviewed By: javache

Differential Revision: D50963933
NickGerleman added a commit to NickGerleman/yoga that referenced this pull request Nov 7, 2023
Summary:
X-link: facebook/react-native#41305


This should not be part of Yoga's API. If benchmarks want to do this, they still can (though I don't know the ones we have for it are super valuable).

Reviewed By: javache

Differential Revision: D50963933
NickGerleman added a commit to NickGerleman/react-native that referenced this pull request Nov 7, 2023
Summary:

X-link: facebook/yoga#1448

This should not be part of Yoga's API. If benchmarks want to do this, they still can (though I don't know the ones we have for it are super valuable).

Reviewed By: javache

Differential Revision: D50963933
NickGerleman added a commit to NickGerleman/react-native that referenced this pull request Nov 7, 2023
Summary:

X-link: facebook/yoga#1448

This should not be part of Yoga's API. If benchmarks want to do this, they still can (though I don't know the ones we have for it are super valuable).

Reviewed By: javache

Differential Revision: D50963933
NickGerleman added a commit to NickGerleman/yoga that referenced this pull request Nov 7, 2023
Summary:
X-link: facebook/react-native#41305


This should not be part of Yoga's API. If benchmarks want to do this, they still can (though I don't know the ones we have for it are super valuable).

Reviewed By: javache

Differential Revision: D50963933
NickGerleman added a commit to NickGerleman/react-native that referenced this pull request Nov 7, 2023
Summary:

X-link: facebook/yoga#1448

This should not be part of Yoga's API. If benchmarks want to do this, they still can (though I don't know the ones we have for it are super valuable).

Reviewed By: javache

Differential Revision: D50963933
NickGerleman added a commit to NickGerleman/yoga that referenced this pull request Nov 7, 2023
Summary:
X-link: facebook/react-native#41305


This should not be part of Yoga's API. If benchmarks want to do this, they still can (though I don't know the ones we have for it are super valuable).

Reviewed By: javache

Differential Revision: D50963933
NickGerleman added a commit to NickGerleman/react-native that referenced this pull request Nov 7, 2023
Summary:

X-link: facebook/yoga#1448

This should not be part of Yoga's API. If benchmarks want to do this, they still can (though I don't know the ones we have for it are super valuable).

Reviewed By: javache

Differential Revision: D50963933
NickGerleman added a commit to NickGerleman/yoga that referenced this pull request Nov 7, 2023
Summary:
X-link: facebook/react-native#41305


This should not be part of Yoga's API. If benchmarks want to do this, they still can (though I don't know the ones we have for it are super valuable).

Reviewed By: javache

Differential Revision: D50963933
NickGerleman added a commit to NickGerleman/react-native that referenced this pull request Nov 7, 2023
Summary:

X-link: facebook/yoga#1448

This should not be part of Yoga's API. If benchmarks want to do this, they still can (though I don't know the ones we have for it are super valuable).

Reviewed By: javache

Differential Revision: D50963933
NickGerleman added a commit to NickGerleman/react-native that referenced this pull request Nov 7, 2023
Summary:

X-link: facebook/yoga#1448

This should not be part of Yoga's API. If benchmarks want to do this, they still can (though I don't know the ones we have for it are super valuable).

Reviewed By: javache

Differential Revision: D50963933
NickGerleman added a commit to NickGerleman/yoga that referenced this pull request Nov 8, 2023
Summary:
X-link: facebook/react-native#41305


This should not be part of Yoga's API. If benchmarks want to do this, they still can (though I don't know the ones we have for it are super valuable).

Reviewed By: javache

Differential Revision: D50963933
facebook-github-bot pushed a commit to facebook/litho that referenced this pull request Nov 8, 2023
Summary:
X-link: facebook/react-native#41305

X-link: facebook/yoga#1448

This should not be part of Yoga's API. If benchmarks want to do this, they still can (though I don't know the ones we have for it are super valuable).

Reviewed By: javache

Differential Revision: D50963933

fbshipit-source-id: 6482bd269928188b6469a358ffde5c4f9f5f9527
facebook-github-bot pushed a commit to facebook/react-native that referenced this pull request Nov 8, 2023
Summary:
Pull Request resolved: #41305

X-link: facebook/yoga#1448

This should not be part of Yoga's API. If benchmarks want to do this, they still can (though I don't know the ones we have for it are super valuable).

Reviewed By: javache

Differential Revision: D50963933

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

This pull request has been merged in 9eb8a62.

Othinn pushed a commit to Othinn/react-native that referenced this pull request Jan 9, 2024
Summary:
Pull Request resolved: facebook#41305

X-link: facebook/yoga#1448

This should not be part of Yoga's API. If benchmarks want to do this, they still can (though I don't know the ones we have for it are super valuable).

Reviewed By: javache

Differential Revision: D50963933

fbshipit-source-id: 6482bd269928188b6469a358ffde5c4f9f5f9527
tungdo194 pushed a commit to tungdo194/rn-test that referenced this pull request Apr 28, 2024
Summary:

X-link: facebook/yoga#1448

This should not be part of Yoga's API. If benchmarks want to do this, they still can (though I don't know the ones we have for it are super valuable).

Reviewed By: javache

Differential Revision: D50963933
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