-
Notifications
You must be signed in to change notification settings - Fork 119
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
Cross-Plattform Crash Reporting: Add Android Support #4484
Conversation
Codecov Report
@@ Coverage Diff @@
## sentry #4484 +/- ##
===========================================
+ Coverage 29.12% 56.33% +27.20%
===========================================
Files 248 12 -236
Lines 13683 600 -13083
Branches 7851 287 -7564
===========================================
- Hits 3985 338 -3647
+ Misses 3920 262 -3658
+ Partials 5778 0 -5778
Flags with carried forward coverage won't be shown. Click here to find out more. Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
@@ -28,7 +28,7 @@ jobs: | |||
run: | | |||
auth_header="$(git config --local --get http.https://github.com/.extraheader)" | |||
git submodule sync --recursive | |||
git -c "http.extraheader=$auth_header" -c protocol.version=2 submodule update --init --force --recursive --depth=1 | |||
git -c "http.extraheader=$auth_header" -c protocol.version=2 submodule update --init --force --recursive --depth=2 |
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.
Why?
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.
the sentry_native repo itself has submodules:
https://github.com/getsentry/sentry-native/tree/master/external
44cffe1
to
7d1bec1
Compare
This adds support to report crashes & errors for android to #4455