Improve config validation

incident-io / catalog-importer

Import data into the incident.io Catalog from a variety of sources.

MIT License

14 stars 4 forks source link

Open lawrencejones opened 1 year ago

lawrencejones commented 1 year ago

The errors are a bit messy. If you fail to parse the file, we print the default template output, which is confusing.

If you use keys that aren't recognised, we'll accept them: this is particularly confusing.

lawrencejones commented 1 year ago

Detection of duplicate type names would be great too.

lawrencejones commented 9 months ago

https://github.com/incident-io/catalog-importer/pull/65 will fix the unrecognised key problem, but several things still left to improve:

General output: it's hard to see the error and the pretty-printing of the config isn't helping anyone.
Duplicate type names.
Other logical config errors.