-
Notifications
You must be signed in to change notification settings - Fork 250
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
Added rules for required for engine and datacapture libraries. #1916
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.
Great change @aditya-07 ! Just two nits
Im also assuming we don't need this in any of our other libraries as they depend on engine and datacapture |
can you also minify the datacapture and the demo app and add a build test? |
revisiting this - discussed with aditya - let's try to merge this to enable minification and code shrinking. some relevant links: https://developer.android.com/build/shrink-code#keep-code, https://developer.android.com/studio/projects/android-library it's not idea if developers have to copy and paste these rules in their own apps. |
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.
with this change can you turn on minification for the demo apps?
…cessary keep rule from catalog's rule file
tested a minified release build of the demo app and got this when i create a patient:
|
Its fixed now with new rules update. |
Thanks Aditya! |
IMPORTANT: All PRs must be linked to an issue (except for extremely trivial and straightforward changes).
Fixes #1901 #2327
Description
Added the proguard rules required for engine and datacapture libs in Demo and catalog app respectively.Added rules to respective
proguard-rules.pro
files for engine and datacapture library as per Android's considerations.This way, R8 automatically applies the provided rules and developers don't have to include the rules explicitly in the app's
proguard-rules
file ( see Library dependencies ).Alternative(s) considered
Publish the rules in wiki, but then users will have to check it with every release to check if rules have been modified.
Type
Choose one: Bug fix
Screenshots (if applicable)
Checklist
./gradlew spotlessApply
and./gradlew spotlessCheck
to check my code follows the style guide of this project../gradlew check
and./gradlew connectedCheck
to test my changes locally.