-
Notifications
You must be signed in to change notification settings - Fork 662
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
Upgrade to compose 1.7 #9894
base: master
Are you sure you want to change the base?
Upgrade to compose 1.7 #9894
Conversation
3 builds increased size
Stripe Identity Example 21.3.1-theme1 (20)
|
Item | Install Size Change | Download Size Change |
---|---|---|
_COROUTINE._BOUNDARY | ⬆️ 39.6 kB | ⬆️ 19.0 kB |
androidx.camera.core.impl.utils.executor.MainThreadExecutor | ⬇️ -30.3 kB | ⬇️ -14.6 kB |
📝 androidx.compose.ui.focus.FocusInteropUtils | ⬆️ 29.1 kB | ⬆️ 14.0 kB |
androidx.compose.ui.input.key.KeyEvent_androidKt | ⬇️ -28.6 kB | ⬇️ -13.7 kB |
🗑 androidx.compose.ui.Modifier$-CC | ⬇️ -27.0 kB | ⬇️ -13.0 kB |
Financial Connections Example 21.3.1 (210301)
com.stripe.android.financialconnections.example
⚖️ Compare build
⏱️ Analyze build performance
Total install size change: ⬆️ 256.2 kB (2.39%)
Total download size change: ⬆️ 124.4 kB (2.28%)
Largest size changes
Item | Install Size Change | Download Size Change |
---|---|---|
📝 androidx.compose.ui.unit.ConstraintsKt | ⬆️ 103.0 kB | ⬆️ 48.5 kB |
📝 androidx.compose.ui.unit.AndroidDensity_androidKt | ⬆️ 80.2 kB | ⬆️ 37.8 kB |
🗑 androidx.room.util.TableInfo$Column$Companion | ⬇️ -78.5 kB | ⬇️ -36.9 kB |
📝 androidx.compose.ui.unit.DensityKt | ⬆️ 78.1 kB | ⬆️ 36.8 kB |
🗑 kotlin.internal.ProgressionUtilKt | ⬇️ -68.1 kB | ⬇️ -32.0 kB |
PaymentSheet Example 21.3.1 (11)
com.stripe.android.paymentsheet.example
⚖️ Compare build
⏱️ Analyze build performance
Total install size change: ⬆️ 12.7 kB (0.08%)
Total download size change: ⬆️ 5.0 kB (0.06%)
Largest size changes
Item | Install Size Change | Download Size Change |
---|---|---|
org.bouncycastle.pqc.crypto.xmss.XMSSNodeUtil | ⬇️ -18.9 kB | ⬇️ -8.7 kB |
com.stripe.android.paymentsheet.paymentdatacollection.ach.USBankA... | ⬇️ -18.1 kB | ⬇️ -8.3 kB |
com.stripe.android.uicore.address.AutocompleteCapableAddressType$... | ⬆️ 13.7 kB | ⬆️ 6.2 kB |
📝 androidx.constraintlayout.core.widgets.analyzer.DependencyNode$Ty... | ⬆️ 12.9 kB | ⬆️ 5.9 kB |
🗑 androidx.constraintlayout.core.widgets.ConstraintAnchor$Type$Enum... | ⬇️ -12.8 kB | ⬇️ -5.9 kB |
🛸 Powered by Emerge Tools
Comment trigger: Size diff threshold of 100.00kB exceeded
Diffuse output:
APK
DEX
ARSC
|
bdba073
to
70eb92a
Compare
4fe9ae2
to
3a35e9d
Compare
77361ee
to
bd8f43c
Compare
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 is bad! Need to figure out how to fix!
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.
Same!
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.
Cursors went missing in these screenshots, but still exist in production!
bd8f43c
to
d427efa
Compare
Summary
I found a workaround for our upgrade issue here: https://issuetracker.google.com/issues/372512084#comment2