Skip to content
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.

Sign up for GitHub

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

perf: reduce APK size with ProGuard enabled #34218

Merged
merged 5 commits into from
Jan 25, 2024
Merged

perf: reduce APK size with ProGuard enabled #34218

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
882b8c6
feat: enable ProGuard
hurali97 Jan 10, 2024
0ae9249
feat: add ProGuard rules
hurali97 Jan 10, 2024
35155fd
fix: add resource keep rules to not remove assets
hurali97 Jan 11, 2024
2479f3e
Merge branch 'main' of https://github.com/callstack-internal/Expensif…
hurali97 Jan 19, 2024
0d6384d
feat: add dontwarn flag to in proguard rules to allow build to succeed
hurali97 Jan 19, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions android/app/src/main/res/raw/keep.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
<?xml version="1.0" encoding="utf-8"?>
<resources xmlns:tools="http://schemas.android.com/tools"
tools:keep="@mipmap/ic_launcher, @drawable/bootsplash*, @drawable/alert_background,
@drawable/ic_launcher*, @drawable/ic_notification*, @drawable/picker_background,
@drawable/rn_edit_text_material" />
Comment on lines +2 to +5
Copy link
Contributor

Choose a reason for hiding this comment

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

This is a small selection of resources, did we get this list from anywhere specific?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I inspected the res directory and all the resources that were used in the codebase by search, were added in keep rule. For example, drawable/datepicker_background.xml wasn't being referenced anywhere, so I didn't add it to the keep rule.

So there's no prescription that says what to keep but anything that's referenced and is being used can be kept. I also tested on build variants like production and adhoc and I get the correct assets as our keep xml file is inherited for each build variant.

https://developer.android.com/build/shrink-code#keep-resources

Screenshot 2024-01-11 at 6 08 06 PM

Copy link
Contributor Author

@hurali97 hurali97 Jan 11, 2024 •
edited
Loading

Choose a reason for hiding this comment

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

Resources in react-native/assets aren't touched by resource shrinker, so that's why they aren't added in the keep rules.

Copy link
Contributor

@Julesssss Julesssss Jan 11, 2024 •
edited
Loading

Choose a reason for hiding this comment

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

I also tested on build variants like production and adhoc and I get the correct assets as our keep xml file is inherited for each build variant.

Okay cool, that's what I was worried about as it's not clear that alternative resourses would be kept. For example: drawable-xxxhdpi/bootsplash_logo.png

I'm just generating another test build to confirm that we're not defaulting to the lower res drawables.

Loading