Skip to content

Generate Kotlin files with enums.py#55631

Closed
NickGerleman wants to merge 1 commit into
facebook:mainfrom
NickGerleman:export-D93555645
Closed

Generate Kotlin files with enums.py#55631
NickGerleman wants to merge 1 commit into
facebook:mainfrom
NickGerleman:export-D93555645

Conversation

@NickGerleman
Copy link
Copy Markdown
Contributor

Summary:
This PR extends enums.py to also write Kotlin files. To introduce this in the least breaking manner, we can gradually do one by one and at the end clean up the Java script.

Migrate YogaDirection to Kotlin.

Changelog: [Internal]

X-link: facebook/yoga#1845

Reviewed By: cortinico

Differential Revision: D93555645

Pulled By: NickGerleman

Summary:
This PR extends enums.py to also write Kotlin files. To introduce this in the least breaking manner, we can gradually do one by one and at the end clean up the Java script.

Migrate YogaDirection to Kotlin.

Changelog: [Internal]

X-link: facebook/yoga#1845

Reviewed By: cortinico

Differential Revision: D93555645

Pulled By: NickGerleman
@meta-codesync
Copy link
Copy Markdown

meta-codesync Bot commented Feb 19, 2026

@NickGerleman has exported this pull request. If you are a Meta employee, you can view the originating Diff in D93555645.

@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Feb 19, 2026
Copy link
Copy Markdown
Contributor

@cortinico cortinico left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review automatically exported from Phabricator review in Meta.

meta-codesync Bot pushed a commit to facebook/yoga that referenced this pull request Feb 20, 2026
Summary:
X-link: facebook/react-native#55631

This PR extends enums.py to also write Kotlin files. To introduce this in the least breaking manner, we can gradually do one by one and at the end clean up the Java script.

Migrate YogaDirection to Kotlin.

Changelog: [Internal]

Pull Request resolved: #1845

Test Plan:
RN
```sh
yarn android
yarn test-android
```

Yoga
```sh
./gradlew :yoga:assembleDebug
```

Reviewed By: cortinico

Differential Revision: D93555645

Pulled By: NickGerleman

fbshipit-source-id: cdddc9850f05cf0782cc4fdb782f22776ab12d31
@meta-codesync meta-codesync Bot closed this in 7e4c994 Feb 20, 2026
@react-native-bot
Copy link
Copy Markdown
Collaborator

This pull request was successfully merged by @mateoguzmana in 7e4c994

When will my fix make it into a release? | How to file a pick request?

@react-native-bot react-native-bot added the Merged This PR has been merged. label Feb 20, 2026
@meta-codesync
Copy link
Copy Markdown

meta-codesync Bot commented Feb 20, 2026

@NickGerleman merged this pull request in 7e4c994.

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. meta-exported p: Facebook Partner: Facebook Partner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants