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

Revert "build(deps): bump @babel/traverse from 7.19.3 to 7.23.2 in /sample-new-architecture (#3352)" #3373

Merged
merged 1 commit into from
Nov 2, 2023

Conversation

krystofwoldrich
Copy link
Member

@krystofwoldrich krystofwoldrich commented Nov 2, 2023

This reverts commit 12427f4.

@babel/traverse dependency bump breaks the sample app bundle.

#skip-changelog

Copy link
Contributor

github-actions bot commented Nov 2, 2023

Android (legacy) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 440.14 ms 468.84 ms 28.70 ms
Size 17.73 MiB 19.85 MiB 2.12 MiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
27ef4ee 317.40 ms 321.70 ms 4.30 ms
457e29f 398.10 ms 421.39 ms 23.29 ms
9c48b2c 349.24 ms 385.96 ms 36.72 ms
8900e1a+dirty 430.68 ms 456.13 ms 25.44 ms
d361d38 354.10 ms 381.69 ms 27.59 ms
9a3ca65+dirty 326.93 ms 330.14 ms 3.21 ms
d197b5c+dirty 338.94 ms 354.87 ms 15.93 ms
0677344 327.74 ms 337.14 ms 9.40 ms
0db0c72 372.12 ms 386.00 ms 13.88 ms
e5c9b8b 409.02 ms 426.66 ms 17.64 ms

App size

Revision Plain With Sentry Diff
27ef4ee 17.73 MiB 19.82 MiB 2.08 MiB
457e29f 17.73 MiB 19.84 MiB 2.10 MiB
9c48b2c 17.73 MiB 19.80 MiB 2.07 MiB
8900e1a+dirty 17.73 MiB 19.75 MiB 2.01 MiB
d361d38 17.73 MiB 19.81 MiB 2.08 MiB
9a3ca65+dirty 17.73 MiB 20.04 MiB 2.31 MiB
d197b5c+dirty 17.73 MiB 20.04 MiB 2.31 MiB
0677344 17.73 MiB 19.81 MiB 2.07 MiB
0db0c72 17.73 MiB 19.75 MiB 2.02 MiB
e5c9b8b 17.73 MiB 19.83 MiB 2.10 MiB

@krystofwoldrich krystofwoldrich merged commit f06c879 into main Nov 2, 2023
45 of 47 checks passed
@krystofwoldrich krystofwoldrich deleted the revert-babel-traverse branch November 2, 2023 10:06
Copy link
Contributor

github-actions bot commented Nov 2, 2023

Android (new) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 350.24 ms 383.62 ms 33.38 ms
Size 7.15 MiB 8.12 MiB 997.78 KiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
d7401ac+dirty 373.98 ms 394.08 ms 20.10 ms
5446992+dirty 371.61 ms 390.00 ms 18.39 ms
52a8031+dirty 330.72 ms 358.76 ms 28.03 ms
575f9da+dirty 337.15 ms 370.47 ms 33.32 ms
15c80ab+dirty 276.38 ms 327.54 ms 51.17 ms
34aba08+dirty 331.79 ms 376.69 ms 44.91 ms
86d6d2c+dirty 267.21 ms 325.24 ms 58.04 ms
b1e8712+dirty 322.55 ms 331.84 ms 9.29 ms
acadc0f+dirty 259.04 ms 304.67 ms 45.63 ms
457e29f+dirty 591.49 ms 612.96 ms 21.47 ms

App size

Revision Plain With Sentry Diff
d7401ac+dirty 7.15 MiB 8.04 MiB 910.85 KiB
5446992+dirty 7.15 MiB 8.12 MiB 999.45 KiB
52a8031+dirty 7.15 MiB 8.09 MiB 965.95 KiB
575f9da+dirty 7.15 MiB 8.10 MiB 979.68 KiB
15c80ab+dirty 7.15 MiB 8.09 MiB 966.13 KiB
34aba08+dirty 7.15 MiB 8.07 MiB 946.13 KiB
86d6d2c+dirty 7.15 MiB 8.09 MiB 962.69 KiB
b1e8712+dirty 7.15 MiB 8.04 MiB 912.27 KiB
acadc0f+dirty 7.15 MiB 8.03 MiB 903.20 KiB
457e29f+dirty 7.15 MiB 8.10 MiB 981.29 KiB

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