Skip to content

chore: Move from using Hybrid SDK subspec to regular Sentry podspec #5451

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.

Sign up for GitHub

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

Jump to bottom
Draft
itaybre wants to merge 2 commits into
base: cocoa-v9
Choose a base branch
from
Draft

chore: Move from using Hybrid SDK subspec to regular Sentry podspec #5451

itaybre wants to merge 2 commits into from

Conversation

@itaybre
Copy link
Contributor

@itaybre itaybre commented Dec 9, 2025

📢 Type of change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring

📜 Description

Replaced the HybridSDK subspec with the regular one.

💡 Motivation and Context

Simplify our maintenance work

💚 How did you test it?

📝 Checklist

  • I added tests to verify 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.
  • All tests passing
  • No breaking changes

🔮 Next steps

@antonis antonis added ready-to-merge Triggers the full CI test suite and removed ready-to-merge Triggers the full CI test suite labels Dec 11, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Dec 11, 2025

Fails
🚫 Pull request is not ready for merge, please add the "ready-to-merge" label to the pull request

Generated by 🚫 dangerJS against 91b4776

@github-actions
Copy link
Contributor

github-actions bot commented Dec 11, 2025

Android (legacy) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 402.58 ms 487.05 ms 84.46 ms
Size 43.75 MiB 48.07 MiB 4.32 MiB

@github-actions
Copy link
Contributor

github-actions bot commented Dec 11, 2025

Android (new) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 403.53 ms 391.39 ms -12.14 ms
Size 43.94 MiB 48.90 MiB 4.96 MiB

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

Reviewers

@alwx alwx Awaiting requested review from alwx alwx will be requested when the pull request is marked ready for review alwx is a code owner

@antonis antonis Awaiting requested review from antonis antonis will be requested when the pull request is marked ready for review antonis is a code owner

@lucas-zimerman lucas-zimerman Awaiting requested review from lucas-zimerman lucas-zimerman will be requested when the pull request is marked ready for review lucas-zimerman is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@itaybre @antonis