Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat/yaml_support #162

Merged
merged 9 commits into from Jun 21, 2022
Merged

feat/yaml_support #162

merged 9 commits into from Jun 21, 2022

Conversation

NeonJarbas
Copy link

@NeonJarbas NeonJarbas commented Jun 17, 2022

allow downstream projects to use yaml as their config format, this can be achieved via ovos.conf

The use case for this PR is "mycroft as lib", this is meant for devs creating their own voice assistants on top of ovos-core, your own assistant can now use yaml if you prefer it!

regular users are not affected by this change, the default config format is still commented json.

allow downstream projects to use yaml as their config format, this can be achieved via ovos.conf
@JarbasAl JarbasAl added the enhancement New feature or request label Jun 17, 2022
@JarbasAl JarbasAl requested a review from NeonDaniel June 17, 2022 15:42
@JarbasAl JarbasAl marked this pull request as ready for review June 17, 2022 15:42
@codecov
Copy link

codecov bot commented Jun 17, 2022

Codecov Report

Merging #162 (2fd56a5) into dev (6ceb058) will increase coverage by 2.77%.
The diff coverage is 43.00%.

@@            Coverage Diff             @@
##              dev     #162      +/-   ##
==========================================
+ Coverage   50.35%   53.12%   +2.77%     
==========================================
  Files         119      152      +33     
  Lines       10077     9815     -262     
==========================================
+ Hits         5074     5214     +140     
+ Misses       5003     4601     -402     
Impacted Files Coverage Δ
mycroft/audio/__main__.py 0.00% <0.00%> (ø)
mycroft/client/enclosure/__main__.py 0.00% <0.00%> (ø)
mycroft/client/enclosure/mark1/arduino.py 0.00% <0.00%> (ø)
mycroft/client/enclosure/mark1/eyes.py 0.00% <0.00%> (ø)
mycroft/client/enclosure/mark1/mouth.py 0.00% <0.00%> (ø)
mycroft/client/speech/__main__.py 0.00% <0.00%> (ø)
mycroft/client/speech/hotword_factory.py 0.00% <0.00%> (-88.89%) ⬇️
mycroft/client/speech/service.py 0.00% <0.00%> (ø)
mycroft/client/speech/silence.py 0.00% <0.00%> (-42.86%) ⬇️
mycroft/client/text/__init__.py 0.00% <0.00%> (ø)
... and 122 more

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 a92a313...2fd56a5. Read the comment docs.

mycroft/configuration/config.py Outdated Show resolved Hide resolved
mycroft/configuration/ovos.py Outdated Show resolved Hide resolved
test/unittests/configuration/mycroft.yml Outdated Show resolved Hide resolved
@NeonDaniel NeonDaniel merged commit 66c54be into OpenVoiceOS:dev Jun 21, 2022
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants