Skip to content

Conversation

@denrase
Copy link
Collaborator

@denrase denrase commented Dec 10, 2025

#skip-changelog

📜 Description

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

💡 Motivation and Context

Closes #6277
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.

@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 17:45
@codecov
Copy link

codecov bot commented Dec 10, 2025

Codecov Report

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

Additional details and impacted files

Impacted file tree graph

@@                         Coverage Diff                          @@
##           feat/add-swift-log-integration     #7021       +/-   ##
====================================================================
+ Coverage                          84.936%   85.084%   +0.147%     
====================================================================
  Files                                 457       454        -3     
  Lines                               27604     27703       +99     
  Branches                            12141     12153       +12     
====================================================================
+ Hits                                23446     23571      +125     
+ Misses                               4118      4087       -31     
- Partials                               40        45        +5     

see 52 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...6006c00. Read the comment docs.

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

@denrase denrase requested a review from itaybre December 15, 2025 13:13
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants