chore(deps): Bump Android SDK to v8.3.0#4490
Conversation
Android (legacy) Performance metrics 🚀
|
| Revision | Plain | With Sentry | Diff |
|---|---|---|---|
| f703940 | 437.08 ms | 449.29 ms | 12.21 ms |
| 43ce2a5 | 335.96 ms | 352.66 ms | 16.70 ms |
| 9ad8411 | 408.49 ms | 400.63 ms | -7.86 ms |
| 9af1463 | 403.72 ms | 386.56 ms | -17.16 ms |
| f107027 | 383.58 ms | 407.79 ms | 24.21 ms |
| f220e4d | 378.24 ms | 431.71 ms | 53.47 ms |
| 48d1309 | 451.34 ms | 470.70 ms | 19.36 ms |
| 8f1e8e1 | 468.76 ms | 462.81 ms | -5.95 ms |
| 13aaa0b | 381.94 ms | 491.94 ms | 110.00 ms |
| 516ff9a | 449.94 ms | 430.71 ms | -19.22 ms |
App size
| Revision | Plain | With Sentry | Diff |
|---|---|---|---|
| f703940 | 17.75 MiB | 20.12 MiB | 2.37 MiB |
| 43ce2a5 | 17.75 MiB | 20.13 MiB | 2.38 MiB |
| 9ad8411 | 17.75 MiB | 20.15 MiB | 2.40 MiB |
| 9af1463 | 17.75 MiB | 20.12 MiB | 2.37 MiB |
| f107027 | 17.75 MiB | 20.15 MiB | 2.40 MiB |
| f220e4d | 17.75 MiB | 20.15 MiB | 2.40 MiB |
| 48d1309 | 17.75 MiB | 20.15 MiB | 2.40 MiB |
| 8f1e8e1 | 17.75 MiB | 20.15 MiB | 2.40 MiB |
| 13aaa0b | 17.75 MiB | 20.13 MiB | 2.38 MiB |
| 516ff9a | 17.75 MiB | 20.13 MiB | 2.38 MiB |
Android (new) Performance metrics 🚀
|
| Revision | Plain | With Sentry | Diff |
|---|---|---|---|
| 43ce2a5+dirty | 412.87 ms | 461.96 ms | 49.08 ms |
| 516ff9a+dirty | 348.83 ms | 389.59 ms | 40.76 ms |
| f703940+dirty | 350.96 ms | 390.84 ms | 39.88 ms |
| 8f1e8e1+dirty | 388.60 ms | 424.30 ms | 35.70 ms |
| 9af1463+dirty | 287.04 ms | 313.50 ms | 26.46 ms |
| f220e4d+dirty | 394.08 ms | 420.40 ms | 26.31 ms |
| f107027+dirty | 390.89 ms | 411.72 ms | 20.83 ms |
| 48d1309+dirty | 395.78 ms | 408.06 ms | 12.28 ms |
| 9ad8411+dirty | 367.73 ms | 371.16 ms | 3.43 ms |
| 13aaa0b+dirty | 382.52 ms | 440.69 ms | 58.17 ms |
App size
| Revision | Plain | With Sentry | Diff |
|---|---|---|---|
| 43ce2a5+dirty | 7.15 MiB | 8.39 MiB | 1.24 MiB |
| 516ff9a+dirty | 7.15 MiB | 8.39 MiB | 1.24 MiB |
| f703940+dirty | 7.15 MiB | 8.39 MiB | 1.24 MiB |
| 8f1e8e1+dirty | 7.15 MiB | 8.41 MiB | 1.26 MiB |
| 9af1463+dirty | 7.15 MiB | 8.39 MiB | 1.24 MiB |
| f220e4d+dirty | 7.15 MiB | 8.41 MiB | 1.26 MiB |
| f107027+dirty | 7.15 MiB | 8.41 MiB | 1.26 MiB |
| 48d1309+dirty | 7.15 MiB | 8.41 MiB | 1.26 MiB |
| 9ad8411+dirty | 7.15 MiB | 8.41 MiB | 1.26 MiB |
| 13aaa0b+dirty | 7.15 MiB | 8.39 MiB | 1.24 MiB |
iOS (new) Performance metrics 🚀
|
| Revision | Plain | With Sentry | Diff |
|---|---|---|---|
| 13aaa0b+dirty | 1227.24 ms | 1233.71 ms | 6.47 ms |
| 9af1463+dirty | 1217.24 ms | 1215.55 ms | -1.69 ms |
| 43ce2a5+dirty | 1222.30 ms | 1210.89 ms | -11.40 ms |
| 9ad8411+dirty | 1229.31 ms | 1225.35 ms | -3.96 ms |
| f220e4d+dirty | 1241.67 ms | 1251.04 ms | 9.37 ms |
| f107027+dirty | 1220.96 ms | 1223.61 ms | 2.65 ms |
| f703940+dirty | 1227.41 ms | 1221.96 ms | -5.45 ms |
| 516ff9a+dirty | 1236.90 ms | 1232.92 ms | -3.98 ms |
| 8f1e8e1+dirty | 1227.39 ms | 1220.58 ms | -6.81 ms |
| 48d1309+dirty | 1227.32 ms | 1227.41 ms | 0.09 ms |
App size
| Revision | Plain | With Sentry | Diff |
|---|---|---|---|
| 13aaa0b+dirty | 3.19 MiB | 4.26 MiB | 1.08 MiB |
| 9af1463+dirty | 3.19 MiB | 4.25 MiB | 1.07 MiB |
| 43ce2a5+dirty | 3.19 MiB | 4.27 MiB | 1.08 MiB |
| 9ad8411+dirty | 3.19 MiB | 4.31 MiB | 1.12 MiB |
| f220e4d+dirty | 3.19 MiB | 4.31 MiB | 1.12 MiB |
| f107027+dirty | 3.19 MiB | 4.31 MiB | 1.12 MiB |
| f703940+dirty | 3.19 MiB | 4.25 MiB | 1.07 MiB |
| 516ff9a+dirty | 3.19 MiB | 4.26 MiB | 1.08 MiB |
| 8f1e8e1+dirty | 3.19 MiB | 4.26 MiB | 1.08 MiB |
| 48d1309+dirty | 3.19 MiB | 4.31 MiB | 1.12 MiB |
iOS (legacy) Performance metrics 🚀
|
| Revision | Plain | With Sentry | Diff |
|---|---|---|---|
| 13aaa0b+dirty | 1227.88 ms | 1225.39 ms | -2.49 ms |
| 9af1463+dirty | 1231.92 ms | 1232.04 ms | 0.12 ms |
| 43ce2a5+dirty | 1228.81 ms | 1221.27 ms | -7.54 ms |
| 9ad8411+dirty | 1226.32 ms | 1234.36 ms | 8.04 ms |
| f220e4d+dirty | 1219.92 ms | 1220.60 ms | 0.68 ms |
| f107027+dirty | 1225.02 ms | 1232.50 ms | 7.48 ms |
| f703940+dirty | 1219.50 ms | 1223.02 ms | 3.52 ms |
| 516ff9a+dirty | 1210.71 ms | 1219.10 ms | 8.40 ms |
| 8f1e8e1+dirty | 1225.67 ms | 1236.16 ms | 10.49 ms |
| 48d1309+dirty | 1228.70 ms | 1223.89 ms | -4.81 ms |
App size
| Revision | Plain | With Sentry | Diff |
|---|---|---|---|
| 13aaa0b+dirty | 2.63 MiB | 3.70 MiB | 1.06 MiB |
| 9af1463+dirty | 2.63 MiB | 3.69 MiB | 1.06 MiB |
| 43ce2a5+dirty | 2.63 MiB | 3.70 MiB | 1.07 MiB |
| 9ad8411+dirty | 2.63 MiB | 3.74 MiB | 1.11 MiB |
| f220e4d+dirty | 2.63 MiB | 3.74 MiB | 1.11 MiB |
| f107027+dirty | 2.63 MiB | 3.74 MiB | 1.11 MiB |
| f703940+dirty | 2.63 MiB | 3.69 MiB | 1.05 MiB |
| 516ff9a+dirty | 2.63 MiB | 3.70 MiB | 1.06 MiB |
| 8f1e8e1+dirty | 2.63 MiB | 3.70 MiB | 1.06 MiB |
| 48d1309+dirty | 2.63 MiB | 3.74 MiB | 1.11 MiB |
d2b541e to
0316113
Compare
# Conflicts: # CHANGELOG.md
lucas-zimerman
left a comment
There was a problem hiding this comment.
The changes are looking good! LGTM!
|
We should evaluate if it's okay to release this in a feature update, or wait for RN SDK v7. Generally we don't have to do a major due to native SDK major, but we should confirm if minSdkVersion bump to 21 and others are not issue for min support RN version 0.65. |
That's a good point @krystofwoldrich 👍
Currently the react native sdk defaults to minSdk 21 (though our gradle is not enforcing that). |
fd074dc to
b8e6e0d
Compare
b8e6e0d to
246e13b
Compare
# Conflicts: # CHANGELOG.md # packages/core/android/build.gradle
246e13b to
6f276a6
Compare
6f276a6 to
ac72161
Compare
93220a5 to
6895033
Compare
1f6a33c to
ebea685
Compare
ebea685 to
d50515c
Compare
edf3857 to
8007f1b
Compare
|
@krystofwoldrich , @lucas-zimerman I've targeted v7 and this is ready for another pass 🙇 |
lucas-zimerman
left a comment
There was a problem hiding this comment.
LGTM! Thank you for the PR!
krystofwoldrich
left a comment
There was a problem hiding this comment.
LGTM 🚀 Thank you!
📢 Type of change
📜 Description
I've opened this PR to fix Android SDK v8.0.0 breakages and deprecations without interfering with the
update-android.shci automation and to get a 2nd eye on the changes.💡 Motivation and Context
Part of #4541
Migration guide from version 7.x to 8.0
Breakages
AndroidProfilerconstructor removed theBuildInfoProvideras part of the minSdkVersion bump to 21getIntegrationshas been replaced withgetIntegrationSetgetPackageshas been replaced withgetPackageSetDeprecations
HubwithScopes💚 How did you test it?
CI, Manual
📝 Checklist
sendDefaultPIIis enabled🔮 Next steps
#skip-changelog