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.

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 Kotlin to 1.6.21, Compose to 1.2.0-rc01 #5162

Merged
merged 2 commits into from Jun 17, 2022
Merged

Conversation

brnunes-stripe
Copy link
Contributor

@brnunes-stripe brnunes-stripe commented Jun 17, 2022

Summary

Upgrading Kotlin was blocked on Compose (#4889), which released a release candidate this week.
This fixes the flickering when moving between text fields in compose, which is visible in all our forms but mostly on the OTP collection UI for Link.

Motivation

Upgrade dependencies.

Testing

  • Added tests
  • Modified tests
  • Manually verified

@github-actions
Copy link
Contributor

github-actions bot commented Jun 17, 2022

Diffuse output:

OLD: paymentsheet-example-release-master.apk (signature: none)
NEW: paymentsheet-example-release-pr.apk (signature: none)

          │            compressed            │           uncompressed           
          ├──────────┬──────────┬────────────┼───────────┬───────────┬──────────
 APK      │ old      │ new      │ diff       │ old       │ new       │ diff     
──────────┼──────────┼──────────┼────────────┼───────────┼───────────┼──────────
      dex │   14 MiB │ 14.8 MiB │ +756.3 KiB │  46.7 MiB │    50 MiB │ +3.4 MiB 
     arsc │  1.7 MiB │  1.7 MiB │   +6.1 KiB │   1.7 MiB │   1.7 MiB │ +6.1 KiB 
 manifest │  3.5 KiB │  3.6 KiB │      +41 B │  16.2 KiB │  16.4 KiB │   +232 B 
      res │  821 KiB │  821 KiB │      -18 B │   1.3 MiB │   1.3 MiB │      0 B 
   native │  2.5 MiB │  2.5 MiB │        0 B │   5.9 MiB │   5.9 MiB │      0 B 
    asset │    3 MiB │    3 MiB │     +378 B │     3 MiB │     3 MiB │   +378 B 
    other │ 79.9 KiB │ 80.2 KiB │     +320 B │ 154.8 KiB │ 154.9 KiB │   +116 B 
──────────┼──────────┼──────────┼────────────┼───────────┼───────────┼──────────
    total │ 22.1 MiB │ 22.8 MiB │ +763.1 KiB │  58.7 MiB │  62.1 MiB │ +3.4 MiB 

         │           raw           │                 unique                 
         ├────────┬────────┬───────┼────────┬────────┬──────────────────────
 DEX     │ old    │ new    │ diff  │ old    │ new    │ diff                 
─────────┼────────┼────────┼───────┼────────┼────────┼──────────────────────
   files │      4 │      4 │     0 │        │        │                      
 strings │ 227206 │ 234063 │ +6857 │ 197693 │ 206906 │ +9213 (+12244 -3031) 
   types │  39658 │  41171 │ +1513 │  36162 │  38151 │ +1989 (+2360 -371)   
 classes │  33524 │  35491 │ +1967 │  33524 │  35491 │ +1967 (+2331 -364)   
 methods │ 203721 │ 209480 │ +5759 │ 195717 │ 202432 │ +6715 (+8916 -2201)  
  fields │ 152435 │ 155187 │ +2752 │ 151408 │ 154228 │ +2820 (+4217 -1397)  

 ARSC    │ old  │ new  │ diff       
─────────┼──────┼──────┼────────────
 configs │  293 │  292 │ -1 (+0 -1) 
 entries │ 5794 │ 5798 │ +4 (+5 -1)
APK
      compressed       │    uncompressed     │                                                                      
──────────┬────────────┼──────────┬──────────┤                                                                      
 size     │ diff       │ size     │ diff     │ path                                                                 
──────────┼────────────┼──────────┼──────────┼──────────────────────────────────────────────────────────────────────
  1.5 MiB │     +1 MiB │  3.7 MiB │ +2.6 MiB │ ∆ classes4.dex                                                       
  3.2 MiB │ -681.7 KiB │  9.9 MiB │ -1.4 MiB │ ∆ classes3.dex                                                       
  6.8 MiB │ +353.3 KiB │ 27.7 MiB │ +2.4 MiB │ ∆ classes.dex                                                        
  3.3 MiB │  +33.5 KiB │  8.7 MiB │ -276 KiB │ ∆ classes2.dex                                                       
  1.7 MiB │   +6.1 KiB │  1.7 MiB │ +6.1 KiB │ ∆ resources.arsc                                                     
  7.7 KiB │     +368 B │  7.5 KiB │   +368 B │ ∆ assets/dexopt/baseline.prof                                        
    215 B │     +215 B │     11 B │    +11 B │ + META-INF/androidx.customview_customview-poolingcontainer.version   
  3.6 KiB │      +41 B │ 16.4 KiB │   +232 B │ ∆ AndroidManifest.xml                                                
    493 B │      +10 B │    361 B │    +10 B │ ∆ assets/dexopt/baseline.profm                                       
    988 B │       -8 B │  2.8 KiB │      0 B │ ∆ res/AE.xml                                                         
    211 B │       +5 B │     11 B │     +5 B │ ∆ META-INF/androidx.compose.foundation_foundation-layout.version     
    197 B │       +5 B │     11 B │     +5 B │ ∆ META-INF/androidx.compose.foundation_foundation.version            
    211 B │       +5 B │     11 B │     +5 B │ ∆ META-INF/androidx.compose.material_material-icons-core.version     
    219 B │       +5 B │     11 B │     +5 B │ ∆ META-INF/androidx.compose.material_material-icons-extended.version 
    203 B │       +5 B │     11 B │     +5 B │ ∆ META-INF/androidx.compose.material_material-ripple.version         
    189 B │       +5 B │     11 B │     +5 B │ ∆ META-INF/androidx.compose.material_material.version                
    203 B │       +5 B │     11 B │     +5 B │ ∆ META-INF/androidx.compose.runtime_runtime-livedata.version         
    203 B │       +5 B │     11 B │     +5 B │ ∆ META-INF/androidx.compose.runtime_runtime-saveable.version         
    185 B │       +5 B │     11 B │     +5 B │ ∆ META-INF/androidx.compose.runtime_runtime.version                  
    183 B │       +5 B │     11 B │     +5 B │ ∆ META-INF/androidx.compose.ui_ui-geometry.version                   
    183 B │       +5 B │     11 B │     +5 B │ ∆ META-INF/androidx.compose.ui_ui-graphics.version                   
    175 B │       +5 B │     11 B │     +5 B │ ∆ META-INF/androidx.compose.ui_ui-text.version                       
    191 B │       +5 B │     11 B │     +5 B │ ∆ META-INF/androidx.compose.ui_ui-tooling-data.version               
    197 B │       +5 B │     11 B │     +5 B │ ∆ META-INF/androidx.compose.ui_ui-tooling-preview.version            
    181 B │       +5 B │     11 B │     +5 B │ ∆ META-INF/androidx.compose.ui_ui-tooling.version                    
    175 B │       +5 B │     11 B │     +5 B │ ∆ META-INF/androidx.compose.ui_ui-unit.version                       
    175 B │       +5 B │     11 B │     +5 B │ ∆ META-INF/androidx.compose.ui_ui-util.version                       
    165 B │       +5 B │     11 B │     +5 B │ ∆ META-INF/androidx.compose.ui_ui.version                            
    205 B │       +5 B │     11 B │     +5 B │ ∆ META-INF/androidx.profileinstaller_profileinstaller.version        
    189 B │       +5 B │     11 B │     +5 B │ ∆ META-INF/androidx.savedstate_savedstate-ktx.version                
    181 B │       +5 B │     11 B │     +5 B │ ∆ META-INF/androidx.savedstate_savedstate.version                    
    686 B │       +3 B │  1.4 KiB │      0 B │ ∆ res/3T.xml                                                         
  1.3 KiB │       -3 B │  3.9 KiB │      0 B │ ∆ res/_n.xml                                                         
  1.8 KiB │       -3 B │ 10.6 KiB │      0 B │ ∆ res/dn.xml                                                         
  1.2 KiB │       -2 B │  3.8 KiB │      0 B │ ∆ res/7
...✂
MANIFEST
@@ -1,3 +1,3 @@
 <manifest
-    android:compileSdkVersion=31
+    android:compileSdkVersion=32
     android:compileSdkVersionCodename=12
@@ -6,3 +6,3 @@
     package=com.stripe.android.paymentsheet.example
-    platformBuildVersionCode=31
+    platformBuildVersionCode=32
     platformBuildVersionName=12
@@ -12,3 +12,3 @@
       android:minSdkVersion=21
-      android:targetSdkVersion=31
+      android:targetSdkVersion=32
       />
@@ -313,2 +313,7 @@
       </intent-filter>
+      <intent-filter>
+        <action
+            android:name=androidx.profileinstaller.action.SKIP_FILE
+            />
+      </intent-filter>
     </receiver>
DEX
STRINGS:

   old    │ new    │ diff                 
  ────────┼────────┼──────────────────────
   197693 │ 206906 │ +9213 (+12244 -3031) 
  + �
  
  �������0�H�@
  + 
  
  
  ��
  ��������0�*�0�H�@
  + 
  
  
  ���
  ��������0�����2�����H�H
  ¢������
  + 
  
  
  ���
  ������0�����*�H�H�¢������
  + 
  
  
  ���
  ������0�2�����0�H�¢������
  + 
  
  
  ���
  ������0�����2�����H�H�¢������
  + 
  
  
  ���
  ������0����������*�H�2�����H�H�¢������
  + 
  
  
  ���
  ������0����������2�����H�2�����H�H�¢������
  + 
  
  
  ���
  ������0���������������*�H�2�����H�2�����H�H�¢������
  + 
  
  
  ���
  ������0���������������2�����H�2�����H�2�����H�H�¢�����	
  + 
  
  
  ���
  ��	���0��������������������*�H�2�����H�2�����H�2�����H�H�¢���	�
  
  + 
  
  
  ���
  ��
  ���0��������������������2�����H�2�����H�2�����H�2��	��H�H�¢���
  ��
  + 
  
  
  ���
  ��������0�2�������0�2�����0�H
  ¢������
  + 
  
  
  ���
  ������0�*�0�H�¢������¨��
  + 
  
  ���
  ��
  ��2�0�ø����
  ��!0�¨��À��
  + 
  
  ���
  ��
  �f�2�0�ø����
  ��!0�¨��À��
  + 
  
  ���
  ��
  �g�2�0�ø����
  ��!0�¨��À��
  + 
  
  ���
  
  ������0�2�����0�H�@
  + 
  
  ���
  ���
  ����0�*�0�H¨��
  + 
  
  ���
  ���
  �����0�*�0�H
  ¢����
  + 
  
  ���
  ���
  ��
  ����0���0�H
  ¢����
  + 
  ���
  
  ���
  ��*���� �2�����H�0�ø����
  ��!0�¨��À��
  + 
  ���
  ��
  ����f� �2�0�:��ø����
  ��!0�¨��À��
  + 
  ���
  ��
  ����f�2�0�R�����0�X¦�¢��������ø����
  ��!0�¨��À��
  + 
  ���
  ���
  ����g� �2�0�:��ø����
  ��!0�¨��À��
  + 
  ���
  ���
  ����Â��2�0�B���¢����¨��
  + 
  ���
  ���
  ����f�2�0�:���ø����
  ��!0�¨��À��
  + 
  ���
  ���
  ����f�2�0�:����ø����
  ��!0�¨��À��
  + �
  
  ��
  
  ���
  �����0�2�����0�H
  ¢����
  + �
  
  ��
  
  ���
  �����0�2�����0�H�¨��
  + �
  
  ��
  
  ���
  ���0�2�����0�H
  ¢����
  + �
  
  ���
  
  ���
  �����0�2�����0�H
  ¢����
  + �
  
  ���
  ���
  ������0�*�����0�0�H�@
  + �
  
  ���
  ���
  ��������0�*�0�H
  ¢������
  + �
  
  ���
  
  ���
  �������0�*�0�H������0�*�0�H�¨��
  + �
  
  ���
  
  ���
  �����0�2�����0�H¨��
  + �
  
  ���
  
  ���
  �����0�*�0�2�����0�H�¨��
  + �
  
  ���
  ���
  ������
  ������0�0�¢��
  ������¨��
  + �
  
  ���
  ���
  ������
  ������0�0�X��¢��
  ������¨��
  + �
  
  ���
  ���
  �����
  ������0�0�X��¢��
  ������������¨��
  + �
  ���
  
  ��
  ����g�*���� �2�0�R�����8X¦�¢��������ø����
  ��!0�¨��À��
  + �
  ���
  ���
  ��
  �g�2
  ������0�0�ø����
  ��!0�¨��À��
  + �
  
  ��
  ���
  ���
  �����0�����2�����0�H
  ¢����
  + �
  
  ���
  
  ���
  ��	5����0�*�0�2����0�8G@GX���¢��
  ��
  ������������������	¨�
  + �
  
  ���
  
  ���
  ��������0�2�����0�ø�¢�������
  ���¨��
  + �
  
  ���
  
  ���
  ������0�2�����0�2�����0�H
  ¢������
  + �
  
  ���
  
  ���
  ������0�2�����0�H�¢������
  + �
  
  ���
  ���
  ���
  � ���0�2�����0�2�����0�2�����0�H¨��
  + �
  
  ���
  ���
  ���
  ���������0�����0���0�H
  ¢����
  + �
  ���
  
  ��
  ���
  ��*���� �2�����H�0�2�����H�0�ø����
  ��!0�¨��À��
  + �
  ���
  ��
  
  ���
  �f�2�0�J�����0�H&ø����
  ��!0�¨��À��
  + �
  ���
  ��
  
  ���
  �g�2�0�J�����0�H&ø����
  ��!0�¨��À��
  + �
  ���
  ��
  
  ���
  �æ���2�0�J�����0�H&ø����
  ��!0�¨��À��
  + �
  ���
  ���
  
  ���
  �f�2�0�J�����0�H&ø����
  ��!0�¨��À��
  + �
  
  ��
  ���
  ���
  ��������0�����2�����0�H
  ¢����¨��
  + �
  
  ���
  ���
  ���
  ��������0�����2�����0�2�����H�H
  ¢������
  + �
  
  ���
  
  ���
  ���
  ���0�2��������0�0�H
  ¢����
  + �
  
  ���
  ���
  ��_
  ������0�����2���������0�����H�0�H�@
  + �
  
  ���
  ���
  ���
  ������0����������2���������H�����H�0�H�¢������
  + �
  
  ���
  ���
  ���
  ������0���������������2�������������H�����H�0�����H�0�H�¢������
  + �
  
  ���
  ���
  ���
  ������0��������������������2*���&��������H�����H�0���������H�����H�0�0�H�¢�����	
  + �
  
  ���
  
  ���
  ���
  ���0�2���������0�����0�0�H
  ¢����
  + �
  
  ���
  ���
  ���
  ��������0�H�-����H�����2�����0�2��������H�0�H��ø�¢�������
  ���20�¨��
  + �
  
  ���
  ���
  ���
  �������0�X��¢��
  �����������������0�X��¢��
  �����	¨�
  
  + �
  ���
  
  ���
  
  ���
  �f�*�����2�����H�0�J���������8�0�H&ø����
  ��!0�¨��À��
  + �
  ���
  ��
  
  ���
  ����f�2�0�J�����0�H&J�����0�H&J�����0�H&ø����
 
...✂
ARSC
CONFIGS:

   old │ new │ diff       
  ─────┼─────┼────────────
   293 │ 292 │ -1 (+0 -1) 
  - color-land
  

ENTRIES:

   old  │ new  │ diff       
  ──────┼──────┼────────────
   5794 │ 5798 │ +4 (+5 -1) 
  + id/consume_window_insets_tag
  + id/is_pooling_container_tag
  + id/pooling_container_listener_holder_tag
  + string/range_end
  + string/range_start
  
  - color/triangle_color

Copy link
Collaborator

@carlosmuvi-stripe carlosmuvi-stripe left a comment

Choose a reason for hiding this comment

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

LGTM!

@brnunes-stripe brnunes-stripe merged commit c108ce4 into master Jun 17, 2022
@brnunes-stripe brnunes-stripe deleted the brnunes/upgrade branch June 17, 2022 20:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants