-
Notifications
You must be signed in to change notification settings - Fork 25k
chore(Android): convert systeminfo module classes to kotlin #47616
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
Conversation
|
Had to suppress the deprecation for SERIAL param in Android Info Module to get the package to compile. However in the Android docs it mentions now that the field is set to #UNKNOWN. Should I replace the Serial field with Unknown ? https://developer.android.com/reference/android/os/Build#SERIAL |
...ative/ReactAndroid/src/main/java/com/facebook/react/modules/systeminfo/AndroidInfoHelpers.kt
Show resolved
Hide resolved
javache
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for contributing here! We actually have a lot of tooling internally to help us do these migrations automatically, but we're doing them one module at a time, since it's easy for regressions to sneak in. This seems pretty safe though.
|
@javache I've added JvmStatic annotation to all the public methods of the object. Let me know if something else needs to be done from my end 👍🏾 |
|
@javache do let me know if something is remaining from my end 🙏 |
|
@javache sorry for the tag again, but anything that needs to be done by me ? |
packages/react-native/ReactAndroid/src/main/java/com/facebook/react/ReactInstanceManager.java
Outdated
Show resolved
Hide resolved
|
@javache has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
|
@javache Anything I need to do to fix these failing tests? 😓 |
Nope, I'm working on some internal build tooling but should be merged soon. |
|
This pull request was successfully merged by @oddlyspaced in 8dc2c90 When will my fix make it into a release? | How to file a pick request? |
Summary:
Saw a lot of PRs and push to convert existing Java code to Kotlin, and wanted to contribute to the cause. This is a starting point for me so I can understand more about the Java to Kotlin conversion process for ReactAndroid package.
Changelog:
Pick one each for the category and type tags:
[ANDROID] [CHANGED] - Migrated systeminfo module code from Java to Kotlin
For more details, see:
https://reactnative.dev/contributing/changelogs-in-pull-requests
Test Plan:
Verified build on local dev environment.