Skip to content

Conversation

itaybre
Copy link
Contributor

@itaybre itaybre commented Sep 25, 2025

Convert SentryDefaultLimits to Swift.
Waiting for #6283, #6281, #6280

#skip-changelog

Copy link

linear bot commented Sep 25, 2025

Copy link

codecov bot commented Sep 25, 2025

Codecov Report

❌ Patch coverage is 95.91837% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 86.749%. Comparing base (7d88965) to head (7ebb98b).
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
Sources/Swift/Networking/DefaultRateLimits.swift 95.918% 2 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@              Coverage Diff              @@
##              main     #6285       +/-   ##
=============================================
+ Coverage   86.742%   86.749%   +0.006%     
=============================================
  Files          438       438               
  Lines        37323     37326        +3     
  Branches     17433     17431        -2     
=============================================
+ Hits         32375     32380        +5     
+ Misses        4902      4900        -2     
  Partials        46        46               
Files with missing lines Coverage Δ
Sources/Sentry/SentryDependencyContainer.m 89.723% <ø> (ø)
Sources/Sentry/SentryEnvelopeRateLimit.m 97.674% <ø> (ø)
Sources/Sentry/SentrySessionReplayIntegration.m 87.108% <ø> (ø)
Sources/Sentry/SentryTransportFactory.m 100.000% <ø> (ø)
Sources/Swift/Networking/DefaultRateLimits.swift 95.918% <95.918%> (ø)

... and 3 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 7d88965...7ebb98b. Read the comment docs.

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