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

❌ Patch coverage is 85.71429% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 85.500%. Comparing base (e648312) to head (738b28a).
⚠️ Report is 200 commits behind head on main.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
Sources/SentrySwiftLog/SentryLogHandler.swift 85.000% 6 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@ Coverage Diff @@ ## main #6593 +/- ## ============================================= + Coverage 85.485% 85.500% +0.014%  ============================================= Files 451 452 +1 Lines 27428 27470 +42 Branches 11934 11938 +4 ============================================= + Hits 23447 23487 +40  - Misses 3697 3940 +243  + Partials 284 43 -241 
Files with missing lines Coverage Δ
SentryTestUtils/ClearTestState.swift 86.000% <100.000%> (+0.583%) ⬆️
Sources/SentrySwiftLog/SentryLogHandler.swift 85.000% <85.000%> (ø)

... and 38 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 e648312...738b28a. Read the comment docs.

Base automatically changed from denrase/swift-log-sentry-poc to main November 5, 2025 14:36
@denrase
Copy link
Collaborator Author

denrase commented Dec 10, 2025

Closing in favour of #7020

@denrase denrase closed this Dec 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants