Closed JarbasAl closed 1 month ago
Happy to see the move to envvars; I have been using these everywhere for Neon, so just need to test for any unexpected side-effects
@coderabbitai review
The ovos_config/meta.py
file has been extensively refactored to improve its handling of configuration paths via environment variables instead of static file checks. The process for loading configurations has been simplified, with deprecated methods removed or marked, allowing for clearer and more efficient management of configuration settings.
File(s) | Summary of Changes |
---|---|
ovos_config/meta.py |
Refactored to focus on OVOS config file locations based on environment variables, streamlined logic, and deprecated methods removed or marked. |
Function Signature Changes | |
get_ovos_config |
No change in parameters. |
save_ovos_config |
No longer accepts any parameters; marked as deprecated. |
get_ovos_default_config_paths |
Removed; returns an empty list. |
is_using_xdg |
Logs a warning and always returns True; marked as deprecated. |
get_xdg_base |
No change in parameters. |
set_xdg_base |
No change in parameters. |
set_config_filename |
No change in parameters. |
get_config_filename |
No change in parameters. |
set_default_config |
No change in parameters. |
Sequence diagrams are omitted as the changes are primarily refactoring and simplifications, with no new features or significant modifications to control flow.
In the realm of OVOS, configs are set,
By variables, our paths we etch.
Deprecated methods, now laid to rest,
Simplified logic, at its best.
To future builds, we now glide,
With streamlined configs, we'll take in stride. 🐇✨
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Attention: Patch coverage is 0%
with 10 lines
in your changes missing coverage. Please review.
Please upload report for BASE (
dev@e047a65
). Learn more about missing BASE report.
Files | Patch % | Lines |
---|---|---|
ovos_config/meta.py | 0.00% | 10 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
move exclusively to env vars, ovos.conf file is of very limited use, also not a standard pattern and very uncommon in other projects, it just causes confusion to users and distro packagers
i dont think any user could make sense of the docs page for this yet https://ovos-technical-manual.openvoiceos.org/config/
(i also simply don't like it and don't want to maintain it :P )
Summary by CodeRabbit
Refactor
Bug Fixes