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

chore(deps): Upgrade e2e tests to RN 0.77.0 #4507

Merged
merged 4 commits into from
Feb 5, 2025
Merged

Conversation

antonis
Copy link
Collaborator

@antonis antonis commented Feb 5, 2025

📢 Type of change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring

⚠️ Based on #4506

📜 Description

Upgrades e2e tests to RN 0.77

💡 Motivation and Context

React Native 0.77 Release

💚 How did you test it?

CI

📝 Checklist

  • I added tests to verify changes
  • No new PII added or SDK only sends newly added PII if sendDefaultPII is enabled
  • I updated the docs if needed.
  • I updated the wizard if needed.
  • All tests passing
  • No breaking changes

🔮 Next steps

#skip-changelog

Copy link
Contributor

github-actions bot commented Feb 5, 2025

Android (new) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 361.10 ms 402.06 ms 40.96 ms
Size 7.15 MiB 8.38 MiB 1.23 MiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
d16beca+dirty 389.49 ms 423.86 ms 34.37 ms
34aba08+dirty 331.79 ms 376.69 ms 44.91 ms
acadc0f+dirty 259.04 ms 304.67 ms 45.63 ms
52a8031+dirty 330.72 ms 358.76 ms 28.03 ms
3261206+dirty 426.43 ms 446.20 ms 19.78 ms
e5c9b8b+dirty 335.40 ms 360.06 ms 24.67 ms
4161236+dirty 429.89 ms 498.74 ms 68.85 ms
c830127+dirty 352.35 ms 388.96 ms 36.61 ms
abb7058+dirty 320.78 ms 324.08 ms 3.30 ms
5571a20+dirty 359.52 ms 389.80 ms 30.28 ms

App size

Revision Plain With Sentry Diff
d16beca+dirty 7.15 MiB 8.37 MiB 1.22 MiB
34aba08+dirty 7.15 MiB 8.07 MiB 946.13 KiB
acadc0f+dirty 7.15 MiB 8.03 MiB 903.20 KiB
52a8031+dirty 7.15 MiB 8.09 MiB 965.95 KiB
3261206+dirty 7.15 MiB 8.38 MiB 1.23 MiB
e5c9b8b+dirty 7.15 MiB 8.10 MiB 980.41 KiB
4161236+dirty 7.15 MiB 8.38 MiB 1.23 MiB
c830127+dirty 7.15 MiB 8.38 MiB 1.23 MiB
abb7058+dirty 7.15 MiB 8.10 MiB 980.40 KiB
5571a20+dirty 7.15 MiB 8.20 MiB 1.05 MiB

Previous results on branch: antonis/rn-0.77-e2e

Startup times

Revision Plain With Sentry Diff
1312f8d+dirty 367.53 ms 425.78 ms 58.25 ms

App size

Revision Plain With Sentry Diff
1312f8d+dirty 7.15 MiB 8.38 MiB 1.23 MiB

Copy link
Contributor

github-actions bot commented Feb 5, 2025

iOS (legacy) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 1243.63 ms 1240.37 ms -3.27 ms
Size 2.63 MiB 3.70 MiB 1.06 MiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
43e66e0+dirty 1226.82 ms 1225.92 ms -0.90 ms
d2c32bb+dirty 1223.69 ms 1229.49 ms 5.80 ms
fe13591+dirty 1208.25 ms 1219.53 ms 11.28 ms
22e31b6+dirty 1253.62 ms 1265.96 ms 12.34 ms
8b86336+dirty 1219.38 ms 1230.73 ms 11.36 ms
c2a4e9b+dirty 1240.10 ms 1239.22 ms -0.88 ms
eb1e19f+dirty 1209.56 ms 1214.94 ms 5.38 ms
80b2ce3+dirty 1265.92 ms 1268.60 ms 2.69 ms
1faf8e3+dirty 1214.87 ms 1222.83 ms 7.97 ms
7f6950b+dirty 1235.53 ms 1227.30 ms -8.23 ms

App size

Revision Plain With Sentry Diff
43e66e0+dirty 2.36 MiB 3.11 MiB 759.78 KiB
d2c32bb+dirty 2.36 MiB 3.08 MiB 737.22 KiB
fe13591+dirty 2.36 MiB 3.10 MiB 752.40 KiB
22e31b6+dirty 2.36 MiB 2.87 MiB 520.67 KiB
8b86336+dirty 2.63 MiB 3.68 MiB 1.05 MiB
c2a4e9b+dirty 2.36 MiB 3.08 MiB 734.00 KiB
eb1e19f+dirty 2.36 MiB 3.08 MiB 737.21 KiB
80b2ce3+dirty 2.36 MiB 2.84 MiB 486.98 KiB
1faf8e3+dirty 2.36 MiB 3.08 MiB 736.75 KiB
7f6950b+dirty 2.36 MiB 3.11 MiB 760.04 KiB

Previous results on branch: antonis/rn-0.77-e2e

Startup times

Revision Plain With Sentry Diff
1312f8d+dirty 1217.12 ms 1216.82 ms -0.31 ms

App size

Revision Plain With Sentry Diff
1312f8d+dirty 2.63 MiB 3.70 MiB 1.06 MiB

Copy link
Contributor

github-actions bot commented Feb 5, 2025

iOS (new) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 1231.17 ms 1220.96 ms -10.21 ms
Size 3.19 MiB 4.26 MiB 1.08 MiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
43e66e0+dirty 1247.39 ms 1253.46 ms 6.07 ms
d2c32bb+dirty 1244.00 ms 1245.77 ms 1.77 ms
fe13591+dirty 1250.69 ms 1246.27 ms -4.43 ms
22e31b6+dirty 1276.55 ms 1278.12 ms 1.57 ms
8b86336+dirty 1236.26 ms 1235.00 ms -1.26 ms
c2a4e9b+dirty 1247.39 ms 1243.04 ms -4.35 ms
eb1e19f+dirty 1229.91 ms 1231.63 ms 1.71 ms
80b2ce3+dirty 1245.12 ms 1262.04 ms 16.92 ms
1faf8e3+dirty 1223.38 ms 1220.56 ms -2.82 ms
7f6950b+dirty 1250.94 ms 1252.92 ms 1.98 ms

App size

Revision Plain With Sentry Diff
43e66e0+dirty 2.92 MiB 3.67 MiB 772.38 KiB
d2c32bb+dirty 2.92 MiB 3.64 MiB 742.84 KiB
fe13591+dirty 2.92 MiB 3.66 MiB 757.71 KiB
22e31b6+dirty 2.92 MiB 3.43 MiB 524.74 KiB
8b86336+dirty 3.19 MiB 4.25 MiB 1.06 MiB
c2a4e9b+dirty 2.92 MiB 3.64 MiB 739.91 KiB
eb1e19f+dirty 2.92 MiB 3.64 MiB 742.82 KiB
80b2ce3+dirty 2.92 MiB 3.40 MiB 492.75 KiB
1faf8e3+dirty 2.92 MiB 3.64 MiB 742.61 KiB
7f6950b+dirty 2.92 MiB 3.67 MiB 772.53 KiB

Previous results on branch: antonis/rn-0.77-e2e

Startup times

Revision Plain With Sentry Diff
1312f8d+dirty 1221.38 ms 1229.02 ms 7.65 ms

App size

Revision Plain With Sentry Diff
1312f8d+dirty 3.19 MiB 4.26 MiB 1.08 MiB

Copy link
Contributor

github-actions bot commented Feb 5, 2025

Android (legacy) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 438.55 ms 458.26 ms 19.71 ms
Size 17.75 MiB 20.11 MiB 2.36 MiB

Baseline results on branch: antonis/rn-0.77-core

Startup times

Revision Plain With Sentry Diff
8357d2f 445.92 ms 432.26 ms -13.66 ms

App size

Revision Plain With Sentry Diff
8357d2f 17.75 MiB 20.11 MiB 2.36 MiB

@antonis antonis marked this pull request as ready for review February 5, 2025 09:55
Base automatically changed from antonis/rn-0.77-core to main February 5, 2025 12:04
@antonis antonis enabled auto-merge (squash) February 5, 2025 12:11
@antonis antonis merged commit a18210a into main Feb 5, 2025
62 of 63 checks passed
@antonis antonis deleted the antonis/rn-0.77-e2e branch February 5, 2025 12:12
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.

2 participants