-
Notifications
You must be signed in to change notification settings - Fork 24.3k
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
Improvements to app bundle size reports #28169
Conversation
|
RNTester.app (iOS): 10657792 bytes |
|
RNTester.app (iOS): 10657792 bytes |
- Generate table for APK sizes - Replace existing comment if possible
b9179bb
to
f5135bd
Compare
eol=$'\n' | ||
size_report="" | ||
size_report="| App | Platform | Engine | Arch | Size (bytes) |$eol" |
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.
suggestion: mb
might be easier to mentally parse then bytes.
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.
@hramos has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
This pull request was successfully merged by @tido64 in 32127e8. When will my fix make it into a release? | Upcoming Releases |
Summary: Make the Android app bundle size report more legible, and update existing reports if possible. ## Changelog [Internal] [Changed] - Update existing app bundle size report if possible [Internal] [Changed] - Generate table for APK sizes report Pull Request resolved: facebook#28169 Test Plan: - App bundle size reports should be updated when possible - Android apk sizes should be reported using a table Reviewed By: cpojer Differential Revision: D20141490 Pulled By: hramos fbshipit-source-id: 3fc1b808b27e893c0a3e4e34eafc39094448799c
Summary
Make the Android app bundle size report more legible, and update existing reports if possible.
Changelog
[Internal] [Changed] - Update existing app bundle size report if possible
[Internal] [Changed] - Generate table for APK sizes report
Test Plan