Skip to content

Add minimum necessary .d.ts files to react-native-codegen#36102

Closed
ZihanChen-MSFT wants to merge 4 commits into
facebook:mainfrom
ZihanChen-MSFT:ts-rncodegen20
Closed

Add minimum necessary .d.ts files to react-native-codegen#36102
ZihanChen-MSFT wants to merge 4 commits into
facebook:mainfrom
ZihanChen-MSFT:ts-rncodegen20

Conversation

@ZihanChen-MSFT
Copy link
Copy Markdown
Contributor

Summary

Add minimum necessary .d.ts files to react-native-codegen.

I found .d.ts files will be copied to lib so I guess no additional script is needed.

Changelog

[GENERAL] [CHANGED] - Add minimum necessary .d.ts files to react-native-codegen

Test Plan

npm run build in packages/react-native-codegen and see all .d.ts files appear in lib.

@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: Microsoft Partner: Microsoft Partner labels Feb 9, 2023
@analysis-bot
Copy link
Copy Markdown

Platform Engine Arch Size (bytes) Diff
android hermes arm64-v8a 8,476,510 +609
android hermes armeabi-v7a 7,798,185 +620
android hermes x86 8,952,630 +630
android hermes x86_64 8,810,552 +831
android jsc arm64-v8a 9,113,976 +297
android jsc armeabi-v7a 8,310,847 +300
android jsc x86 9,165,652 +308
android jsc x86_64 9,424,659 +521

Base commit: 6d1667c
Branch: main

@cortinico cortinico requested a review from cipolleschi February 9, 2023 11:47
Copy link
Copy Markdown
Contributor

@cipolleschi cipolleschi left a comment

Choose a reason for hiding this comment

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

Looks good to me!

@facebook-github-bot
Copy link
Copy Markdown
Contributor

@cipolleschi has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

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

@cipolleschi merged this pull request in be3845a.

@ZihanChen-MSFT ZihanChen-MSFT deleted the ts-rncodegen20 branch February 13, 2023 22:28
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. Merged This PR has been merged. p: Microsoft Partner: Microsoft Partner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants