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.095%. Comparing base (6106fab) to head (1bade41).
✅ 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                          85.084%   85.095%   +0.010%     
====================================================================
  Files                                 454       454               
  Lines                               27703     27703               
  Branches                            12151     12153        +2     
====================================================================
+ Hits                                23571     23574        +3     
+ Misses                               4086      4085        -1     
+ Partials                               46        44        -2     

see 9 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 6106fab...1bade41. 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

Comment on lines +47 to +49
public override init() {
super.init()
}
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this needed?

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