search

jellyfin / jellyfin-expo

Jellyfin Mobile for iOS
https://jellyfin.org
Mozilla Public License 2.0
318 stars 68 forks source link

Update codecov config to be less strict on coverage changes #300

Closed thornbill closed 3 years ago

thornbill commented 3 years ago

Updates the codecov config to allow for larger changes in code coverage before failing a PR. It seems codecov made some change to how they are computing coverage which is causing all current PRs to fail due to a perceived drop in coverage.

sonarcloud[bot] commented 3 years ago

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

codecov[bot] commented 3 years ago

Codecov Report

Merging #300 (9693d20) into master (0ec9042) will decrease coverage by 1.38%. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #300      +/-   ##
==========================================
- Coverage   34.33%   32.95%   -1.39%     
==========================================
  Files          35       37       +2     
  Lines         664      701      +37     
  Branches      132      136       +4     
==========================================
+ Hits          228      231       +3     
- Misses        395      425      +30     
- Partials       41       45       +4
Impacted Files Coverage Δ
i18n.js 100.00% <0.00%> (ø)
App.js 0.00% <0.00%> (ø)

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 0ec9042...9693d20. Read the comment docs.