search

OpenVoiceOS / ovos_skill_manager

skill installer for OVOS
Apache License 2.0
5 stars 6 forks source link

Troubleshoot ovos-utils version compat. #80

Closed NeonDaniel closed 2 years ago

codecov[bot] commented 2 years ago

Codecov Report

Merging #80 (5a61ad0) into master (ba89385) will increase coverage by 2.27%. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #80      +/-   ##
==========================================
+ Coverage   56.38%   58.65%   +2.27%     
==========================================
  Files          33       33              
  Lines        2680     2680              
==========================================
+ Hits         1511     1572      +61     
+ Misses       1169     1108      -61
Impacted Files Coverage Δ
ovos_skills_manager/github/__init__.py 72.76% <0.00%> (+4.06%) :arrow_up:
ovos_skills_manager/requirements.py 30.43% <0.00%> (+7.24%) :arrow_up:
ovos_skills_manager/github/api.py 42.28% <0.00%> (+10.29%) :arrow_up:

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 cccfcc2...5a61ad0. Read the comment docs.

NeonDaniel commented 2 years ago

I missed the relevant bug, but the change speaks for itself.

The most recent couple of unit test failures show the issue