DOC/TST/NF/RF: another collection of docstrings and general improvements - Githubissues

psyinfra / onyo

text-based inventory system on top of git

ISC License

3 stars 5 forks source link

DOC/TST/NF/RF: another collection of docstrings and general improvements #621

Closed aqw closed 3 months ago

aqw commented 3 months ago

This PR again focuses on docstrings, and then makes adjustments/cleanups that i encountered while focusing on them.

Docstring improvements:

all of command_utils.py
about 20% of commands.py

Non-docstring improvements include:

sanitize_args_config() -> allowed_config_args()
onyo_edit()'s behavior has been changed to match other onyo commands
normalization of type checking imports
normalization of some help text string declaration

Works towards:

489
492
493

aqw commented 3 months ago

Thanks for the review! I have fixed everything and pushed.