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.

Sign up for GitHub

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

Jump to bottom

chore(deps): Bump reach native to 0.77.1 #4560

Merged
merged 5 commits into from
Feb 19, 2025
Merged

chore(deps): Bump reach native to 0.77.1 #4560

merged 5 commits into from
Feb 19, 2025

Conversation

antonis
Copy link
Collaborator

@antonis antonis commented Feb 18, 2025

📢 Type of change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring

📜 Description

Bumps React Native from 0.77.0 to 0.77.1 for core, e2e and samples/react-native

Refs:

💡 Motivation and Context

Issues related to RN 0.77.1 like #4553

💚 How did you test it?

Manual, 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

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Feb 18, 2025 •
edited
Loading

Android (legacy) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 426.85 ms 416.44 ms -10.41 ms
Size 17.75 MiB 20.11 MiB 2.37 MiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
94fc925 446.39 ms 484.80 ms 38.41 ms
c398f67 449.64 ms 461.38 ms 11.74 ms
4161236 442.79 ms 433.02 ms -9.77 ms
abb7058 370.27 ms 389.58 ms 19.31 ms
8b86336 424.98 ms 413.06 ms -11.92 ms
77680ec 482.22 ms 475.07 ms -7.16 ms
76d1baf+dirty 335.72 ms 355.52 ms 19.80 ms
70e6261 482.65 ms 495.70 ms 13.05 ms
cdf2f33 469.46 ms 462.17 ms -7.29 ms
2ec71da 438.14 ms 460.46 ms 22.32 ms

App size

Revision Plain With Sentry Diff
94fc925 17.75 MiB 20.11 MiB 2.37 MiB
c398f67 17.73 MiB 19.94 MiB 2.21 MiB
4161236 17.75 MiB 20.11 MiB 2.37 MiB
abb7058 17.73 MiB 19.83 MiB 2.10 MiB
8b86336 17.75 MiB 20.11 MiB 2.36 MiB
77680ec 17.74 MiB 20.08 MiB 2.34 MiB
76d1baf+dirty 17.73 MiB 20.04 MiB 2.31 MiB
70e6261 17.73 MiB 19.94 MiB 2.21 MiB
cdf2f33 17.74 MiB 20.08 MiB 2.34 MiB
2ec71da 17.73 MiB 20.10 MiB 2.37 MiB

Previous results on branch: antonis/bump-rn0.77.1

Startup times

Revision Plain With Sentry Diff
6fe3879 418.19 ms 442.19 ms 24.00 ms

App size

Revision Plain With Sentry Diff
6fe3879 17.75 MiB 20.11 MiB 2.37 MiB

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Feb 18, 2025 •
edited
Loading

iOS (legacy) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 1231.92 ms 1228.29 ms -3.63 ms
Size 2.63 MiB 3.74 MiB 1.11 MiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
baa882f+dirty 1218.00 ms 1227.04 ms 9.04 ms
70e6261+dirty 1220.09 ms 1230.04 ms 9.95 ms
2dc8f22+dirty 1224.88 ms 1212.40 ms -12.47 ms
13f280b+dirty 1229.25 ms 1226.10 ms -3.15 ms
52c0562+dirty 1219.34 ms 1221.25 ms 1.91 ms
800171e+dirty 1229.96 ms 1232.87 ms 2.91 ms
c398f67+dirty 1219.67 ms 1225.66 ms 5.99 ms
db44eaf+dirty 1227.09 ms 1230.65 ms 3.56 ms
ed3d77e+dirty 1214.61 ms 1205.83 ms -8.78 ms
1c65324+dirty 1235.17 ms 1235.08 ms -0.09 ms

App size

Revision Plain With Sentry Diff
baa882f+dirty 2.36 MiB 3.08 MiB 731.91 KiB
70e6261+dirty 2.36 MiB 3.03 MiB 680.42 KiB
2dc8f22+dirty 2.63 MiB 3.69 MiB 1.05 MiB
13f280b+dirty 2.36 MiB 3.10 MiB 753.43 KiB
52c0562+dirty 2.36 MiB 3.14 MiB 793.36 KiB
800171e+dirty 2.63 MiB 3.69 MiB 1.05 MiB
c398f67+dirty 2.36 MiB 3.04 MiB 696.27 KiB
db44eaf+dirty 2.36 MiB 3.10 MiB 755.81 KiB
ed3d77e+dirty 2.36 MiB 3.10 MiB 753.32 KiB
1c65324+dirty 2.36 MiB 3.04 MiB 698.64 KiB

Previous results on branch: antonis/bump-rn0.77.1

Startup times

Revision Plain With Sentry Diff
6fe3879+dirty 1208.76 ms 1218.92 ms 10.16 ms

App size

Revision Plain With Sentry Diff
6fe3879+dirty 2.63 MiB 3.74 MiB 1.11 MiB

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Feb 18, 2025 •
edited
Loading

Android (new) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 392.24 ms 403.56 ms 11.31 ms
Size 7.15 MiB 8.38 MiB 1.23 MiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
8fe7c9d+dirty 363.91 ms 429.93 ms 66.03 ms
77680ec+dirty 393.50 ms 426.68 ms 33.18 ms
61310e1+dirty 463.51 ms 511.09 ms 47.57 ms
d197b5c+dirty 258.75 ms 313.61 ms 54.86 ms
62a750b+dirty 370.78 ms 376.73 ms 5.96 ms
800171e+dirty 432.15 ms 549.63 ms 117.48 ms
79976dd+dirty 373.25 ms 404.64 ms 31.39 ms
12427f4+dirty 379.48 ms 400.92 ms 21.44 ms
c2a4e9b+dirty 392.94 ms 474.55 ms 81.61 ms
7143e6f+dirty 338.60 ms 374.92 ms 36.32 ms

App size

Revision Plain With Sentry Diff
8fe7c9d+dirty 7.15 MiB 8.38 MiB 1.23 MiB
77680ec+dirty 7.15 MiB 8.35 MiB 1.20 MiB
61310e1+dirty 7.15 MiB 8.36 MiB 1.21 MiB
d197b5c+dirty 7.15 MiB 8.09 MiB 962.72 KiB
62a750b+dirty 7.15 MiB 8.21 MiB 1.06 MiB
800171e+dirty 7.15 MiB 8.38 MiB 1.23 MiB
79976dd+dirty 7.15 MiB 8.38 MiB 1.23 MiB
12427f4+dirty 7.15 MiB 8.12 MiB 997.78 KiB
c2a4e9b+dirty 7.15 MiB 8.34 MiB 1.19 MiB
7143e6f+dirty 7.15 MiB 8.38 MiB 1.23 MiB

Previous results on branch: antonis/bump-rn0.77.1

Startup times

Revision Plain With Sentry Diff
6fe3879+dirty 392.43 ms 379.76 ms -12.67 ms

App size

Revision Plain With Sentry Diff
6fe3879+dirty 7.15 MiB 8.38 MiB 1.23 MiB

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Feb 18, 2025 •
edited
Loading

iOS (new) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 1232.12 ms 1229.51 ms -2.61 ms
Size 3.19 MiB 4.31 MiB 1.12 MiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
baa882f+dirty 1235.48 ms 1229.02 ms -6.46 ms
70e6261+dirty 1224.90 ms 1231.02 ms 6.12 ms
2dc8f22+dirty 1215.43 ms 1206.68 ms -8.75 ms
13f280b+dirty 1217.73 ms 1229.08 ms 11.35 ms
52c0562+dirty 1233.94 ms 1226.29 ms -7.65 ms
800171e+dirty 1222.94 ms 1235.82 ms 12.88 ms
c398f67+dirty 1227.31 ms 1230.00 ms 2.69 ms
db44eaf+dirty 1238.49 ms 1236.56 ms -1.93 ms
ed3d77e+dirty 1246.19 ms 1244.00 ms -2.19 ms
1c65324+dirty 1239.71 ms 1239.86 ms 0.15 ms

App size

Revision Plain With Sentry Diff
baa882f+dirty 2.92 MiB 3.64 MiB 738.56 KiB
70e6261+dirty 2.92 MiB 3.59 MiB 686.11 KiB
2dc8f22+dirty 3.19 MiB 4.25 MiB 1.06 MiB
13f280b+dirty 2.92 MiB 3.66 MiB 758.67 KiB
52c0562+dirty 2.92 MiB 3.69 MiB 794.15 KiB
800171e+dirty 3.19 MiB 4.25 MiB 1.06 MiB
c398f67+dirty 2.92 MiB 3.60 MiB 701.89 KiB
db44eaf+dirty 2.92 MiB 3.66 MiB 761.15 KiB
ed3d77e+dirty 2.92 MiB 3.66 MiB 758.57 KiB
1c65324+dirty 2.92 MiB 3.61 MiB 705.56 KiB

Previous results on branch: antonis/bump-rn0.77.1

Startup times

Revision Plain With Sentry Diff
6fe3879+dirty 1220.46 ms 1216.07 ms -4.39 ms

App size

Revision Plain With Sentry Diff
6fe3879+dirty 3.19 MiB 4.31 MiB 1.12 MiB

@antonis antonis marked this pull request as ready for review February 18, 2025 14:54
@antonis antonis marked this pull request as draft February 18, 2025 14:55
@antonis antonis marked this pull request as ready for review February 18, 2025 15:42
krystofwoldrich
krystofwoldrich approved these changes Feb 19, 2025
View reviewed changes
Copy link
Member

@krystofwoldrich krystofwoldrich left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

@krystofwoldrich krystofwoldrich enabled auto-merge (squash) February 19, 2025 10:13
@krystofwoldrich krystofwoldrich merged commit 13b68d9 into main Feb 19, 2025
65 checks passed
@krystofwoldrich krystofwoldrich deleted the antonis/bump-rn0.77.1 branch February 19, 2025 10:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@krystofwoldrich krystofwoldrich krystofwoldrich approved these changes

@lucas-zimerman lucas-zimerman Awaiting requested review from lucas-zimerman lucas-zimerman is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@antonis @krystofwoldrich