-
-
Notifications
You must be signed in to change notification settings - Fork 440
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
Add API to set custom sentry-native sdk name #2704
Conversation
|
Performance metrics 🚀
|
Codecov ReportPatch and project coverage have no change.
Additional details and impacted files@@ Coverage Diff @@
## main #2704 +/- ##
=========================================
Coverage 81.05% 81.05%
Complexity 4427 4427
=========================================
Files 345 345
Lines 16356 16356
Branches 2219 2219
=========================================
Hits 13258 13258
Misses 2170 2170
Partials 928 928 ☔ View full report in Codecov by Sentry. |
@krystofwoldrich can you take a look if it's fine for you? we provide |
@marandaneto @krystofwoldrich if there's no objection from your side I'll merge this PR at the end of today. |
📜 Description
Update sentry-native to 0.6.2
Add
SentryAndroidOptions.setNativeSdkName
Fixes #2678
💡 Motivation and Context
💚 How did you test it?
Added some unit tests. Ideally we would have some integration test for this as well, but there's nothing setup as of now.
📝 Checklist
sendDefaultPII
is enabled.🔮 Next steps
#skip-changelog