Skip to content

Conversation

@denrase
Copy link
Collaborator

@denrase denrase commented Oct 29, 2025

📜 Description

💡 Motivation and Context

💚 How did you test it?

📝 Checklist

You have to check all boxes before merging:

  • I added tests to verify the 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.
  • Review from the native team if needed.
  • No breaking change or entry added to the changelog.
  • No breaking change for hybrid SDKs or communicated to hybrid SDKs.

@denrase denrase changed the title Structured Logs: Add SntryCocoaLumberjack integration Structured Logs: Add SentryCocoaLumberjack integration Oct 29, 2025
@codecov
Copy link

codecov bot commented Oct 29, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.500%. Comparing base (476a3d6) to head (738b28a).
✅ All tests successful. No failed tests found.

Additional details and impacted files

Impacted file tree graph

@@                        Coverage Diff                         @@
##           denrase/swift-log-sentry-poc     #6593       +/-   ##
==================================================================
+ Coverage                        85.435%   85.500%   +0.065%     
==================================================================
  Files                               452       452               
  Lines                             27470     27470               
  Branches                          11932     11938        +6     
==================================================================
+ Hits                              23469     23487       +18     
- Misses                             3714      3940      +226     
+ Partials                            287        43      -244     

see 42 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 476a3d6...738b28a. 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.

2 participants