-
Notifications
You must be signed in to change notification settings - Fork 294
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
Update instructions for Android and our sample app #900
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #900 +/- ##
=========================================
Coverage 86.99% 86.99%
Complexity 1958 1958
=========================================
Files 77 77
Lines 6319 6319
Branches 1223 1223
=========================================
Hits 5497 5497
Misses 418 418
Partials 404 404 ☔ View full report in Codecov by Sentry. |
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.
Documentation updates LGTM.
Not saying that we should do this, but perhaps if this breaks more often we can consider adding an integration test CI job that sets this up and use regex to check that the output contains the expected errors 😃
Yeah I'm hoping it doesn't come to that 🙂 But if needed, that would be a good way to test for regressions. |
This reverts commit 115d683.
Fixes #891. It's hard to add a regression test for this, but I manually tested that with the new configuration, NullAway errors are detected in the sample app. After this lands, I'd like to update the readme further to refer to this particular commit, so users can see the changes we needed to make.