Skip to content

Conversation

@denrase
Copy link
Collaborator

@denrase denrase commented Dec 10, 2025

#skip-changelog

📜 Description

  • Adds the SentryCocoaLumberjack integration to the 3rd-party-integrations/SentryCocoaLumberjack folder
  • Skipping changelog as this is not available to users through this repo

💡 Motivation and Context

Closes #6278
Relates to #6902

💚 How did you test it?

  • Runs pure swift tests against the current Package.swift version
  • This loads pre-build artefacts, so we always test against already released versions.

📝 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.
@codecov
Copy link

codecov bot commented Dec 10, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.088%. Comparing base (18e2765) to head (c7ff522).
✅ All tests successful. No failed tests found.

Additional details and impacted files

Impacted file tree graph

@@ Coverage Diff @@ ## feat/add-swift-log-integration #7020 +/- ## ==================================================================== + Coverage 84.936% 85.088% +0.151%  ==================================================================== Files 457 454 -3 Lines 27604 27703 +99 Branches 12141 12153 +12 ==================================================================== + Hits 23446 23572 +126  + Misses 4118 4087 -31  - Partials 40 44 +4 

see 49 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 18e2765...c7ff522. Read the comment docs.

@denrase denrase added the ready-to-merge Use this label to trigger all PR workflows label Dec 10, 2025
@denrase denrase marked this pull request as ready for review December 10, 2025 16:03
Copy link
Member

@philprime philprime left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-to-merge Use this label to trigger all PR workflows

3 participants