-
Notifications
You must be signed in to change notification settings - Fork 51
SES-1971 : Enable minification #1566
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
base: dev
Are you sure you want to change the base?
SES-1971 : Enable minification #1566
Conversation
app/build.gradle.kts
Outdated
| isDefault = true | ||
| isMinifyEnabled = false | ||
|
|
||
| isMinifyEnabled = true |
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.
We shouldn't have anything set on debug as it should run without minification and obfuscation
|
There are some repeated lines for jackson and a few redundant declarations throughout where -keep already does a lot of the work repeated in other lines. |
app/build.gradle.kts
Outdated
| getByName("debug") { | ||
| isDefault = true | ||
| isMinifyEnabled = false | ||
| proguardFiles( |
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.
This can be removed in debug since we are not using R8 on debug builds
SES-1971
Initial minification and proguard rules setup.