Conversation
f69a854 to
5d17a59
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #229 +/- ##
==========================================
- Coverage 79.50% 79.32% -0.19%
==========================================
Files 55 55
Lines 3221 3221
Branches 535 535
==========================================
- Hits 2561 2555 -6
- Misses 613 617 +4
- Partials 47 49 +2 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
In the current state the patch reduced test coverage in some part of the code. |
|
@rotu Friendly ping. |
|
Thanks for the ping. |
5d9c766 to
a9f8179
Compare
|
@dirk-thomas Friendly ping. |
dirk-thomas
left a comment
There was a problem hiding this comment.
Thanks for improving the tests.
I will just wait for clicking merge until #215 is good to be merged (to avoid the temporary drop of coverage between them).
|
Please rebase / resolve conflict to be able to re-review and merge this patch. |
|
@rotu Please rebase this branch in a way that the pull requests only shows commits related to this change. |
8c4e3b3 to
4912050
Compare
Oops. Nothing like rebasing on the wrong master. Blech. Let me try one more time... Success! |
4912050 to
04713ee
Compare
Package identifications test was doing way too much in one test to follow.
Split into multiple, more targeted tests and added some additional coverage