codecov[bot]
commented
11 months ago Codecov Report
Merging #230 (ccdc30e) into master (4926f42) will increase coverage by
5.84%. The diff coverage isn/a.
@@ Coverage Diff @@
## master #230 +/- ##
==========================================
+ Coverage 81.05% 86.88% +5.84%
==========================================
Files 39 38 -1
Lines 1171 983 -188
Branches 180 88 -92
==========================================
- Hits 949 854 -95
+ Misses 161 100 -61
+ Partials 61 29 -32 | Files Changed | Coverage Δ | |
|---|---|---|
| ...r/src/main/kotlin/it/skrape/fetcher/HttpFetcher.kt | 66.67% <ø> (+15.56%) |
:arrow_up: |
christian-draeger
First I just wanted to clean up a bit :) That's why I created
http-fetcher/extensions.ktto keep the structure aligned with async-fetcher's structure. And that is also why I removed some unused code.After I created
http-fetcher/extensions.kt, kover always ran into anArrayIndexOutOfBoundsException. A quick look at the changelog shows that the problem is fixed in version 0.6.1. So I updated to 0.6.1 according to the Migration GuideAfter that, it was natural to go one step further and update directly to the latest version 0.7.3. There is also a Migration Guide for this.
Which projects are included in the merged coverage report? All except
examples/*(see the root build.gradle.kts)Merged Xml Report Will still be stored where the gradle pipeline expects it to be:
./build/reports/kover/report.xmlMerged Html Report Will no longer be created. Let me know if it is needed somewhere and I will add it again.
I did some tests but that doesn't mean that everything works as it should on the Gitlab CI server. Therefore, I gladly accept feedback and adjust again if necessary.