search

OpenVoiceOS / ovos-config

OVOS configuration manager library
Apache License 2.0
2 stars 10 forks source link

fix: unbound local var #127

Closed mikejgray closed 3 months ago

mikejgray commented 3 months ago

chore: resolve issues discovered by linter

Closes #118

Local results (Mac OS):

❯ ovos-config set -k blacklisted_skills -v skill-ovos-wikipedia.openvoiceos
❯ cat ~/.config/mycroft/mycroft.conf | grep -5 wikipedia
    }
  },
  "skills": {
    "blacklisted_skills": [
      "mycroft-joke.mycroftai",
      "skill-ovos-wikipedia.openvoiceos"
    ],
    "installer": {
      "allow_pip": false,
      "break_system_packages": false
    }
codecov[bot] commented 3 months ago

Codecov Report

Attention: Patch coverage is 0% with 11 lines in your changes missing coverage. Please review.

Please upload report for BASE (dev@f1e1c6d). Learn more about missing BASE report.

Files Patch % Lines
ovos_config/models.py 0.00% 6 Missing :warning:
ovos_config/__main__.py 0.00% 4 Missing :warning:
ovos_config/utils.py 0.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## dev #127 +/- ## ===================================== Coverage ? 0.00% ===================================== Files ? 10 Lines ? 817 Branches ? 0 ===================================== Hits ? 0 Misses ? 817 Partials ? 0 ```

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