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

[v9]: Bump JS SDK to 9.5.0 #2784

Merged
merged 3 commits into from
Mar 18, 2025
Merged

[v9]: Bump JS SDK to 9.5.0 #2784

merged 3 commits into from
Mar 18, 2025

Conversation

buenaflor
Copy link
Contributor

@buenaflor buenaflor commented Mar 11, 2025

📜 Description

💡 Motivation and Context

Closes #2694

💚 How did you test it?

📝 Checklist

  • I reviewed submitted code
  • 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
  • All tests passing
  • No breaking changes

🔮 Next steps

Copy link

codecov bot commented Mar 11, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.73%. Comparing base (60f3a6d) to head (cd31632).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2784      +/-   ##
==========================================
+ Coverage   88.62%   89.73%   +1.11%     
==========================================
  Files         263       91     -172     
  Lines        8766     2953    -5813     
==========================================
- Hits         7769     2650    -5119     
+ Misses        997      303     -694     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@buenaflor buenaflor marked this pull request as ready for review March 11, 2025 16:08
Base automatically changed from v9 to main March 12, 2025 15:14
Copy link
Contributor

Android Performance metrics 🚀

  Plain With Sentry Diff
Startup time 451.62 ms 508.43 ms 56.81 ms
Size 6.44 MiB 7.56 MiB 1.12 MiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
2966d88 365.71 ms 422.50 ms 56.79 ms
74e1fdd 398.61 ms 418.43 ms 19.82 ms
9f9f94f 331.04 ms 368.92 ms 37.88 ms
bf4aed7 311.24 ms 365.66 ms 54.42 ms
0b7de9a 455.15 ms 552.73 ms 97.59 ms
ec4c9da 476.70 ms 514.04 ms 37.34 ms
8f88a49 364.98 ms 413.78 ms 48.80 ms
90a08ea 477.25 ms 534.10 ms 56.85 ms
955541a 360.53 ms 452.10 ms 91.57 ms
a1a1545 295.21 ms 351.16 ms 55.95 ms

App size

Revision Plain With Sentry Diff
2966d88 6.33 MiB 7.27 MiB 954.11 KiB
74e1fdd 6.33 MiB 7.27 MiB 954.12 KiB
9f9f94f 5.94 MiB 6.95 MiB 1.01 MiB
bf4aed7 6.06 MiB 7.03 MiB 997.04 KiB
0b7de9a 6.46 MiB 7.48 MiB 1.02 MiB
ec4c9da 6.52 MiB 7.59 MiB 1.06 MiB
8f88a49 6.34 MiB 7.30 MiB 979.60 KiB
90a08ea 6.49 MiB 7.55 MiB 1.06 MiB
955541a 6.27 MiB 7.20 MiB 957.75 KiB
a1a1545 5.94 MiB 6.96 MiB 1.02 MiB

Previous results on branch: v9-dep/bump-js-version

Startup times

Revision Plain With Sentry Diff
469415c 453.00 ms 515.58 ms 62.58 ms

App size

Revision Plain With Sentry Diff
469415c 6.44 MiB 7.56 MiB 1.11 MiB

@buenaflor buenaflor merged commit bd69c1b into main Mar 18, 2025
59 of 61 checks passed
@buenaflor buenaflor deleted the v9-dep/bump-js-version branch March 18, 2025 11:31
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.

[v9]: Bump Sentry JS dependency to v9 for JS SDK Integration
2 participants