Skip to content

Conversation

@marandaneto
Copy link
Contributor

@marandaneto marandaneto commented Mar 8, 2021

📜 Description

Bump: sentry-android to v4.3.0

💡 Motivation and Context

Kotlin nullability and sentry-native improvements

💚 How did you test it?

📝 Checklist

  • I reviewed submitted code
  • I added tests to verify changes
  • I updated the docs if needed
  • All tests passing
  • No breaking changes

🔮 Next steps

@marandaneto marandaneto requested a review from a team March 8, 2021 10:05
@codecov-io
Copy link

codecov-io commented Mar 8, 2021

Codecov Report

Merging #343 (b858bc6) into main (b1856b6) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #343   +/-   ##
=======================================
  Coverage   89.60%   89.60%           
=======================================
  Files          51       51           
  Lines        1636     1636           
=======================================
  Hits         1466     1466           
  Misses        170      170           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b1856b6...b858bc6. Read the comment docs.

Copy link
Member

@philipphofmann philipphofmann left a comment

Choose a reason for hiding this comment

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

ktlint is complaining a bit. Apart from that LGTM.

-keep class io.sentry.flutter.** { *; }

# To ensure that stack traces is unambiguous
# https://developer.android.com/studio/build/shrink-code#decode-stack-trace
Copy link
Member

Choose a reason for hiding this comment

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

Thanks for the important comment 💯

Copy link
Member

@bruno-garcia bruno-garcia left a comment

Choose a reason for hiding this comment

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

LGTM

@marandaneto marandaneto merged commit 5eb036c into main Mar 9, 2021
@marandaneto marandaneto deleted the bump/android-4.3.0 branch March 9, 2021 09:29
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.

5 participants