Skip to content

Comments

Ref(CI): Add android sdk version check#5686

Open
lucas-zimerman wants to merge 5 commits intomainfrom
lz/check-gradle
Open

Ref(CI): Add android sdk version check#5686
lucas-zimerman wants to merge 5 commits intomainfrom
lz/check-gradle

Conversation

@lucas-zimerman
Copy link
Collaborator

@lucas-zimerman lucas-zimerman commented Feb 19, 2026 •
edited
Loading

📢 Type of change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring

📜 Description

This add an extra CI check for the current android version to check if it matches with the one used by the gradle plugin.

💡 Motivation and Context

#5682
The goal is to avoid situations like this in the future.

💚 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

@lucas-zimerman @claude
add android sdk version check
ed1082f 
Co-authored-by: Claude <noreply@anthropic.com>
@github-actions
Copy link
Contributor

github-actions bot commented Feb 19, 2026 •
edited
Loading

Semver Impact of This PR

⚪ None (no version bump detected)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).

  • Ref(CI): Add android sdk version check by lucas-zimerman in #5686

🤖 This preview updates automatically when you update the PR.

@lucas-zimerman
Copy link
Collaborator Author

lucas-zimerman commented Feb 19, 2026

image current situation.

@lucas-zimerman lucas-zimerman added the ready-to-merge Triggers the full CI test suite label Feb 19, 2026
@lucas-zimerman lucas-zimerman marked this pull request as ready for review February 20, 2026 11:59
This was referenced Feb 20, 2026
Open
Open
@github-actions
Copy link
Contributor

github-actions bot commented Feb 20, 2026

iOS (legacy) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 1230.15 ms 1225.81 ms -4.34 ms
Size 3.38 MiB 4.78 MiB 1.40 MiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
ec14be7+dirty 1234.64 ms 1245.54 ms 10.90 ms
b3b5b0d+dirty 1227.71 ms 1239.50 ms 11.79 ms
af9331b+dirty 1233.61 ms 1230.50 ms -3.11 ms
8a868fe+dirty 1221.50 ms 1230.78 ms 9.28 ms
90afdd3+dirty 1233.90 ms 1240.90 ms 7.00 ms
7b02433+dirty 1209.39 ms 1210.90 ms 1.51 ms
8d89cc9+dirty 1222.92 ms 1239.43 ms 16.51 ms
f17e051+dirty 1193.71 ms 1223.51 ms 29.80 ms
cd5f27f+dirty 1214.87 ms 1218.72 ms 3.85 ms
e2fa43d+dirty 1216.04 ms 1232.65 ms 16.61 ms

App size

Revision Plain With Sentry Diff
ec14be7+dirty 2.63 MiB 3.98 MiB 1.34 MiB
b3b5b0d+dirty 2.63 MiB 3.91 MiB 1.28 MiB
af9331b+dirty 2.63 MiB 3.91 MiB 1.28 MiB
8a868fe+dirty 3.38 MiB 4.60 MiB 1.22 MiB
90afdd3+dirty 2.63 MiB 3.99 MiB 1.35 MiB
7b02433+dirty 3.38 MiB 4.60 MiB 1.22 MiB
8d89cc9+dirty 2.63 MiB 3.96 MiB 1.33 MiB
f17e051+dirty 3.41 MiB 4.67 MiB 1.25 MiB
cd5f27f+dirty 3.38 MiB 4.78 MiB 1.39 MiB
e2fa43d+dirty 2.63 MiB 3.81 MiB 1.18 MiB

@github-actions
Copy link
Contributor

github-actions bot commented Feb 20, 2026

Android (legacy) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 492.90 ms 545.87 ms 52.97 ms
Size 43.75 MiB 48.42 MiB 4.67 MiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
ff5a06a+dirty 405.97 ms 439.24 ms 33.27 ms
8ff81c0+dirty 574.58 ms 611.44 ms 36.86 ms
8d0a325+dirty 406.48 ms 415.02 ms 8.54 ms
6c36ba5+dirty 385.48 ms 442.36 ms 56.88 ms
817b2c1+dirty 471.91 ms 489.55 ms 17.64 ms
136effd+dirty 398.57 ms 494.91 ms 96.34 ms
95aaf8a 437.89 ms 419.45 ms -18.44 ms
01d24f6+dirty 362.50 ms 362.23 ms -0.27 ms
6bb8427+dirty 360.48 ms 400.04 ms 39.56 ms
1853710 555.47 ms 556.59 ms 1.12 ms

App size

Revision Plain With Sentry Diff
ff5a06a+dirty 43.75 MiB 48.05 MiB 4.29 MiB
8ff81c0+dirty 43.75 MiB 48.05 MiB 4.29 MiB
8d0a325+dirty 43.75 MiB 48.08 MiB 4.33 MiB
6c36ba5+dirty 43.75 MiB 48.41 MiB 4.66 MiB
817b2c1+dirty 43.75 MiB 48.40 MiB 4.64 MiB
136effd+dirty 43.75 MiB 47.99 MiB 4.23 MiB
95aaf8a 17.75 MiB 19.68 MiB 1.93 MiB
01d24f6+dirty 17.75 MiB 19.74 MiB 2.00 MiB
6bb8427+dirty 43.75 MiB 48.08 MiB 4.33 MiB
1853710 17.75 MiB 19.68 MiB 1.94 MiB

@github-actions
Copy link
Contributor

github-actions bot commented Feb 20, 2026

iOS (new) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 1219.42 ms 1222.68 ms 3.26 ms
Size 3.38 MiB 4.78 MiB 1.40 MiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
ec14be7+dirty 1229.62 ms 1230.53 ms 0.91 ms
b3b5b0d+dirty 1235.16 ms 1238.61 ms 3.45 ms
af9331b+dirty 1230.08 ms 1233.24 ms 3.16 ms
8a868fe+dirty 1206.85 ms 1215.04 ms 8.19 ms
90afdd3+dirty 1216.17 ms 1225.55 ms 9.38 ms
7b02433+dirty 1202.43 ms 1210.90 ms 8.47 ms
8d89cc9+dirty 1232.35 ms 1228.53 ms -3.82 ms
f17e051+dirty 1217.85 ms 1219.96 ms 2.11 ms
cd5f27f+dirty 1207.66 ms 1213.48 ms 5.82 ms
e2fa43d+dirty 1224.43 ms 1235.40 ms 10.98 ms

App size

Revision Plain With Sentry Diff
ec14be7+dirty 3.19 MiB 4.54 MiB 1.36 MiB
b3b5b0d+dirty 3.19 MiB 4.48 MiB 1.29 MiB
af9331b+dirty 3.19 MiB 4.48 MiB 1.29 MiB
8a868fe+dirty 3.38 MiB 4.60 MiB 1.22 MiB
90afdd3+dirty 3.19 MiB 4.55 MiB 1.37 MiB
7b02433+dirty 3.38 MiB 4.60 MiB 1.22 MiB
8d89cc9+dirty 3.19 MiB 4.53 MiB 1.35 MiB
f17e051+dirty 3.41 MiB 4.67 MiB 1.25 MiB
cd5f27f+dirty 3.38 MiB 4.78 MiB 1.39 MiB
e2fa43d+dirty 3.19 MiB 4.38 MiB 1.19 MiB

@github-actions
Copy link
Contributor

github-actions bot commented Feb 20, 2026

Android (new) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 420.18 ms 440.16 ms 19.98 ms
Size 43.94 MiB 49.28 MiB 5.34 MiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
55b77fc+dirty 410.46 ms 414.11 ms 3.65 ms
c7f264b+dirty 356.98 ms 407.46 ms 50.48 ms
90e7cb3+dirty 415.17 ms 458.48 ms 43.31 ms
d1fd647+dirty 374.46 ms 409.51 ms 35.05 ms
7091004+dirty 377.76 ms 402.11 ms 24.35 ms
69602ce+dirty 375.37 ms 405.28 ms 29.91 ms
1c38acd+dirty 411.84 ms 470.18 ms 58.34 ms
3e0a5f9+dirty 379.92 ms 450.96 ms 71.04 ms
bb4ea33+dirty 360.73 ms 375.02 ms 14.29 ms
0b64753+dirty 358.55 ms 429.16 ms 70.61 ms

App size

Revision Plain With Sentry Diff
55b77fc+dirty 43.94 MiB 48.82 MiB 4.88 MiB
c7f264b+dirty 7.15 MiB 8.41 MiB 1.26 MiB
90e7cb3+dirty 43.94 MiB 48.85 MiB 4.91 MiB
d1fd647+dirty 7.15 MiB 8.43 MiB 1.28 MiB
7091004+dirty 43.94 MiB 48.81 MiB 4.88 MiB
69602ce+dirty 7.15 MiB 8.41 MiB 1.26 MiB
1c38acd+dirty 43.94 MiB 48.91 MiB 4.97 MiB
3e0a5f9+dirty 7.15 MiB 8.42 MiB 1.27 MiB
bb4ea33+dirty 43.94 MiB 48.91 MiB 4.97 MiB
0b64753+dirty 7.15 MiB 8.42 MiB 1.27 MiB

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@alwx alwx Awaiting requested review from alwx alwx is a code owner

@antonis antonis Awaiting requested review from antonis antonis is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

ready-to-merge Triggers the full CI test suite

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@lucas-zimerman