Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix(sample-e2e): Fix type errors missing sentry/core and afterAll #4564

Merged
merged 1 commit into from
Feb 20, 2025

Conversation

krystofwoldrich
Copy link
Member

@krystofwoldrich krystofwoldrich commented Feb 19, 2025

Fixes type errors after merge in main to capture-app-start-errors.

https://github.com/getsentry/sentry-react-native/actions/runs/13392121407/job/37404373063?pr=4472

#skip-changelog

Copy link
Contributor

Android (legacy) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 429.83 ms 458.19 ms 28.36 ms
Size 17.75 MiB 20.11 MiB 2.37 MiB

Baseline results on branch: capture-app-start-errors

Startup times

Revision Plain With Sentry Diff
1a89392 412.59 ms 424.69 ms 12.10 ms
18a5066 427.89 ms 436.47 ms 8.57 ms
b6d933c 442.52 ms 461.82 ms 19.30 ms
208f4af 439.23 ms 427.31 ms -11.92 ms
5625ce7 476.74 ms 488.24 ms 11.50 ms
b75148e 440.04 ms 421.36 ms -18.68 ms
555070f 438.67 ms 428.30 ms -10.37 ms
454f971 436.27 ms 478.76 ms 42.48 ms

App size

Revision Plain With Sentry Diff
1a89392 17.75 MiB 20.11 MiB 2.37 MiB
18a5066 17.75 MiB 20.11 MiB 2.37 MiB
b6d933c 17.75 MiB 20.11 MiB 2.37 MiB
208f4af 17.75 MiB 20.11 MiB 2.37 MiB
5625ce7 17.75 MiB 20.11 MiB 2.37 MiB
b75148e 17.75 MiB 20.11 MiB 2.37 MiB
555070f 17.75 MiB 20.11 MiB 2.37 MiB
454f971 17.75 MiB 20.11 MiB 2.37 MiB

Copy link
Contributor

iOS (legacy) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 1214.35 ms 1233.29 ms 18.94 ms
Size 2.63 MiB 3.74 MiB 1.11 MiB

Baseline results on branch: capture-app-start-errors

Startup times

Revision Plain With Sentry Diff
555070f+dirty 1213.59 ms 1217.79 ms 4.20 ms
208f4af+dirty 1209.44 ms 1217.13 ms 7.70 ms
5625ce7+dirty 1226.98 ms 1217.77 ms -9.21 ms
b75148e+dirty 1221.53 ms 1220.85 ms -0.68 ms
1a89392+dirty 1220.12 ms 1216.69 ms -3.43 ms
18a5066+dirty 1218.00 ms 1212.27 ms -5.73 ms
b6d933c+dirty 1217.71 ms 1220.58 ms 2.87 ms
454f971+dirty 1220.55 ms 1225.82 ms 5.27 ms

App size

Revision Plain With Sentry Diff
555070f+dirty 2.63 MiB 3.69 MiB 1.05 MiB
208f4af+dirty 2.63 MiB 3.69 MiB 1.05 MiB
5625ce7+dirty 2.63 MiB 3.70 MiB 1.06 MiB
b75148e+dirty 2.63 MiB 3.69 MiB 1.06 MiB
1a89392+dirty 2.63 MiB 3.70 MiB 1.06 MiB
18a5066+dirty 2.63 MiB 3.70 MiB 1.06 MiB
b6d933c+dirty 2.63 MiB 3.70 MiB 1.06 MiB
454f971+dirty 2.63 MiB 3.74 MiB 1.11 MiB

Copy link
Contributor

iOS (new) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 1223.83 ms 1229.00 ms 5.17 ms
Size 3.19 MiB 4.31 MiB 1.12 MiB

Baseline results on branch: capture-app-start-errors

Startup times

Revision Plain With Sentry Diff
555070f+dirty 1223.61 ms 1227.57 ms 3.96 ms
208f4af+dirty 1213.08 ms 1223.82 ms 10.73 ms
5625ce7+dirty 1219.73 ms 1223.80 ms 4.07 ms
b75148e+dirty 1202.72 ms 1212.04 ms 9.32 ms
1a89392+dirty 1229.00 ms 1234.78 ms 5.78 ms
18a5066+dirty 1244.20 ms 1251.54 ms 7.34 ms
b6d933c+dirty 1228.02 ms 1235.32 ms 7.30 ms
454f971+dirty 1233.51 ms 1239.18 ms 5.67 ms

App size

Revision Plain With Sentry Diff
555070f+dirty 3.19 MiB 4.25 MiB 1.06 MiB
208f4af+dirty 3.19 MiB 4.25 MiB 1.06 MiB
5625ce7+dirty 3.19 MiB 4.26 MiB 1.08 MiB
b75148e+dirty 3.19 MiB 4.25 MiB 1.07 MiB
1a89392+dirty 3.19 MiB 4.26 MiB 1.08 MiB
18a5066+dirty 3.19 MiB 4.26 MiB 1.08 MiB
b6d933c+dirty 3.19 MiB 4.26 MiB 1.08 MiB
454f971+dirty 3.19 MiB 4.31 MiB 1.12 MiB

@krystofwoldrich krystofwoldrich marked this pull request as ready for review February 19, 2025 15:23
Copy link
Collaborator

@antonis antonis left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

Copy link
Contributor

Android (new) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 364.29 ms 383.20 ms 18.91 ms
Size 7.15 MiB 8.38 MiB 1.23 MiB

Baseline results on branch: capture-app-start-errors

Startup times

Revision Plain With Sentry Diff
1a89392+dirty 425.56 ms 530.65 ms 105.09 ms
18a5066+dirty 370.06 ms 414.10 ms 44.04 ms
454f971+dirty 439.04 ms 423.88 ms -15.16 ms
208f4af+dirty 346.93 ms 402.77 ms 55.84 ms
555070f+dirty 388.25 ms 424.44 ms 36.19 ms
b6d933c+dirty 398.43 ms 456.62 ms 58.19 ms
b75148e+dirty 428.91 ms 461.26 ms 32.35 ms
5625ce7+dirty 358.15 ms 416.65 ms 58.50 ms

App size

Revision Plain With Sentry Diff
1a89392+dirty 7.15 MiB 8.38 MiB 1.23 MiB
18a5066+dirty 7.15 MiB 8.38 MiB 1.23 MiB
454f971+dirty 7.15 MiB 8.38 MiB 1.23 MiB
208f4af+dirty 7.15 MiB 8.38 MiB 1.23 MiB
555070f+dirty 7.15 MiB 8.38 MiB 1.23 MiB
b6d933c+dirty 7.15 MiB 8.38 MiB 1.23 MiB
b75148e+dirty 7.15 MiB 8.38 MiB 1.23 MiB
5625ce7+dirty 7.15 MiB 8.38 MiB 1.23 MiB

@krystofwoldrich krystofwoldrich merged commit 0bf6636 into capture-app-start-errors Feb 20, 2025
70 of 71 checks passed
@krystofwoldrich krystofwoldrich deleted the kw-fix-bad-merge branch February 20, 2025 09:04
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants