Skip to content

Conversation

noahsmartin
Copy link
Contributor

#skip-changelog

Copy link

codecov bot commented Aug 22, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.724%. Comparing base (d1c0538) to head (428b0bb).
✅ All tests successful. No failed tests found.

Additional details and impacted files

Impacted file tree graph

@@              Coverage Diff              @@
##              main     #5988       +/-   ##
=============================================
- Coverage   86.760%   86.724%   -0.036%     
=============================================
  Files          424       424               
  Lines        36723     36723               
  Branches     17365     17364        -1     
=============================================
- Hits         31861     31848       -13     
- Misses        4815      4829       +14     
+ Partials        47        46        -1     
Files with missing lines Coverage Δ
...rations/SessionReplay/RRWeb/SentryRRWebEvent.swift 100.000% <ø> (ø)
Sources/Swift/SentryAppState.swift 97.058% <ø> (ø)

... and 6 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d1c0538...428b0bb. Read the comment docs.

Copy link
Contributor

github-actions bot commented Aug 22, 2025

Performance metrics 🚀

  Plain With Sentry Diff
Startup time 1232.11 ms 1257.87 ms 25.76 ms
Size 23.75 KiB 928.88 KiB 905.14 KiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
605fa27 1226.31 ms 1251.35 ms 25.05 ms
cc02c0d 1234.69 ms 1247.15 ms 12.46 ms
7f2f69c 1237.61 ms 1266.96 ms 29.35 ms
4d264fa 1223.48 ms 1246.91 ms 23.44 ms
409a607 1229.57 ms 1251.45 ms 21.88 ms
80a5166 1224.49 ms 1251.29 ms 26.80 ms
029a804 1219.65 ms 1241.96 ms 22.30 ms
04ff3ec 1220.71 ms 1253.86 ms 33.15 ms
aadbffe 1228.73 ms 1251.59 ms 22.86 ms
73c9712 1238.57 ms 1260.38 ms 21.80 ms

App size

Revision Plain With Sentry Diff
605fa27 23.75 KiB 908.03 KiB 884.28 KiB
cc02c0d 23.75 KiB 912.37 KiB 888.62 KiB
7f2f69c 23.75 KiB 913.38 KiB 889.63 KiB
4d264fa 23.74 KiB 874.07 KiB 850.33 KiB
409a607 23.74 KiB 874.08 KiB 850.33 KiB
80a5166 23.75 KiB 904.53 KiB 880.78 KiB
029a804 23.75 KiB 928.11 KiB 904.36 KiB
04ff3ec 23.75 KiB 880.26 KiB 856.52 KiB
aadbffe 23.75 KiB 912.77 KiB 889.02 KiB
73c9712 23.75 KiB 908.01 KiB 884.26 KiB

Previous results on branch: removeSentrySerializablePublicAPI

Startup times

Revision Plain With Sentry Diff
91c6aea 1223.82 ms 1250.40 ms 26.59 ms

App size

Revision Plain With Sentry Diff
91c6aea 23.74 KiB 926.60 KiB 902.86 KiB

@noahsmartin noahsmartin force-pushed the removeSentrySerializablePublicAPI branch from 3de946b to 428b0bb Compare August 28, 2025 23:18
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.

1 participant