psyinfra / onyo

text-based inventory system on top of git
ISC License
3 stars 5 forks source link

Assess StrictYAML as a parser #618

Open aqw opened 3 months ago

aqw commented 3 months ago

The motivation to choose ruamel was support to retain comments (aka: round-trip mode). Otherwise, it's still got loads of YAML footguns and not terribly helpful error messages.

These experiences have inspired the StrictYAML project. It also supports comments, and improved error messages (#587).

It may also relate to validation (#256) due to typing, etc.

Note from Ben in a review, regarding validate_yaml():

If we only collect the file paths here, we are throwing away the actual error. I'd argue not all YAML errors are super obvious, so it would be nice if we find a nice way to give the actual failure per file.