Skip to content

Conversation

@romtsn
Copy link
Member

@romtsn romtsn commented Nov 6, 2025

#skip-changelog

we were getting the sentry-android module installed from maven central, therefore the PR changes made in its submodules were not reflected in the size analysis breakdown

Copy link
Contributor

@chromy chromy left a comment

Choose a reason for hiding this comment

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

lgtm

@github-actions
Copy link
Contributor

github-actions bot commented Nov 6, 2025

Performance metrics 🚀

  Plain With Sentry Diff
Startup time 321.83 ms 369.25 ms 47.42 ms
Size 1.58 MiB 2.12 MiB 551.77 KiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
d217708 375.27 ms 415.68 ms 40.41 ms
fcec2f2 314.96 ms 373.66 ms 58.70 ms
ee747ae 405.43 ms 485.70 ms 80.28 ms
bdbe1f4 380.66 ms 464.44 ms 83.78 ms
b750b96 421.25 ms 444.09 ms 22.84 ms
674d437 355.28 ms 504.18 ms 148.90 ms
ee747ae 415.92 ms 470.15 ms 54.23 ms
3d205d0 352.15 ms 432.53 ms 80.38 ms
27d7cf8 306.76 ms 366.66 ms 59.90 ms
b3d8889 420.46 ms 453.71 ms 33.26 ms

App size

Revision Plain With Sentry Diff
d217708 1.58 MiB 2.10 MiB 532.97 KiB
fcec2f2 1.58 MiB 2.12 MiB 551.50 KiB
ee747ae 1.58 MiB 2.10 MiB 530.95 KiB
bdbe1f4 1.58 MiB 2.11 MiB 538.88 KiB
b750b96 1.58 MiB 2.10 MiB 533.20 KiB
674d437 1.58 MiB 2.10 MiB 530.94 KiB
ee747ae 1.58 MiB 2.10 MiB 530.95 KiB
3d205d0 1.58 MiB 2.10 MiB 532.97 KiB
27d7cf8 1.58 MiB 2.12 MiB 549.42 KiB
b3d8889 1.58 MiB 2.10 MiB 535.07 KiB

@romtsn romtsn merged commit ae7fed0 into main Nov 6, 2025
62 of 64 checks passed
@romtsn romtsn deleted the rz/fix/size-analysis branch November 6, 2025 20:25
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.

4 participants