epics-containers / edge-containers-cli

command line shortcuts for epics containers developers
Apache License 2.0
3 stars 1 forks source link

check for Chart.yaml instead of optional values.yaml #117

Closed gilesknap closed 3 months ago

gilesknap commented 3 months ago

Not all services require a values.yaml and the error still mentioned IOCs.

gilesknap commented 3 months ago

yeah that was a hasty fix :-)

codecov[bot] commented 3 months ago

Codecov Report

Attention: Patch coverage is 50.00000% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 86.80%. Comparing base (08ecdb2) to head (01a0e31).

Files Patch % Lines
src/edge_containers_cli/utils.py 50.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #117 +/- ## ======================================= Coverage 86.80% 86.80% ======================================= Files 14 14 Lines 788 788 ======================================= Hits 684 684 Misses 104 104 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.