Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make HybridPublic folder headers public in Sentry.framework #4384

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

denrase
Copy link
Collaborator

@denrase denrase commented Oct 1, 2024

📜 Description

We want to support SPM in our sentry flutter framework. In order to do this, we need the SentryHybridSDK classes, which we expose though Cocoapods, to also be available in our SPM package.

This PR is fairly straight forward, as it only exposes the headers as public in the Sentry target.

Another option would be to create a separate target SentryHybrid, the same way as SentrySwiftUI for example. I'm open to suggestions.

💡 Motivation and Context

Unblocks getsentry/sentry-dart#2280

💚 How did you test it?

📝 Checklist

You have to check all boxes before merging:

  • I reviewed the submitted code.
  • 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.
  • 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.

🔮 Next steps

Copy link

github-actions bot commented Oct 1, 2024

Fails
🚫 Please consider adding a changelog entry for the next release.
Messages
📖 Do not forget to update Sentry-docs with your feature once the pull request gets approved.

Instructions and example for changelog

Please add an entry to CHANGELOG.md to the "Unreleased" section. Make sure the entry includes this PR's number.

Example:

## Unreleased

- Make `HybridPublic` folder headers public in `Sentry.framework` ([#4384](https://github.com/getsentry/sentry-cocoa/pull/4384))

If none of the above apply, you can opt out of this check by adding #skip-changelog to the PR description.

Generated by 🚫 dangerJS against 0f20fc7

@denrase denrase mentioned this pull request Oct 1, 2024
11 tasks
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.

1 participant