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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
SentryOptions#merge is now public and can be used to load ExternalOptions #2088
Conversation
@@ -1710,7 +1710,7 @@ private SentryOptions(final boolean empty) { | |||
* | |||
* @param options options loaded from external locations | |||
*/ | |||
void merge(final @NotNull ExternalOptions options) { | |||
public void merge(final @NotNull ExternalOptions options) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@marandaneto do you see any harm in making this public?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If there's a use case on the logging integrations, I'm fine with it, this is not used on Android, so your call.
Co-authored-by: Manoel Aranda Neto <5731772+marandaneto@users.noreply.github.com>
Codecov Report
@@ Coverage Diff @@
## main #2088 +/- ##
=========================================
Coverage 81.11% 81.11%
Complexity 3232 3232
=========================================
Files 230 230
Lines 11850 11850
Branches 1572 1572
=========================================
Hits 9612 9612
Misses 1652 1652
Partials 586 586
Continue to review full report at Codecov.
|
馃摐 Description
Allows developers to deserialize
ExternalOptions
and then merge them intoSentryOptions
.馃挕 Motivation and Context
Closes #2086
馃挌 How did you test it?
馃摑 Checklist
馃敭 Next steps