-
Notifications
You must be signed in to change notification settings - Fork 8
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
Add Troubleshooting Android section #172
Conversation
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.
Looks good overall! Thank you @dangeross
Added some notes on wording, suggested a new heap space value & a section recommendation about when code obfuscation may kick in and cause issues re: JNA library.
|
||
Starting with this 1.8.20 release, the Kotlin K2 compiler has a preview version of the serialization plugin. It's recommended for users to upgrade their Kotlin version to at least 1.8.20 or [set up the serialization plugin](https://github.com/Kotlin/kotlinx.serialization?tab=readme-ov-file#setup) on their projects explicitly with older versions. | ||
|
||
## JNA library |
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.
JNA library code obfuscation issue may not be apparent until users compile their apps on "release"
mode with maximum optimization and minimal footprint size that'll be used when deploying & publishing their app.
I think we should add a warning about this in this section.
Co-authored-by: Erdem Yerebasmaz <[email protected]>
f0e4d39
to
3fdf938
Compare
Co-authored-by: Erdem Yerebasmaz <[email protected]>
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.
LGTM! 🎉
Fixes #171