-
Notifications
You must be signed in to change notification settings - Fork 2
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
Print Errors for config validation #94
Comments
lincmba
added a commit
that referenced
this issue
Dec 22, 2023
lincmba
added a commit
that referenced
this issue
Dec 22, 2023
pld
added a commit
that referenced
this issue
Jan 8, 2024
fixes ##94 Co-authored-by: Peter Lubell-Doughtie <[email protected]>
pld
added a commit
that referenced
this issue
Jan 8, 2024
* Print config errors instead of exiting fixes ##94 * Add config tests Succeeds #98 Tests should run against the changes on this PR #99 fixes #93 --------- Co-authored-by: Peter Lubell-Doughtie <[email protected]>
Closing this since PR is merged in here #99 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Currently the config validation works like any regular JSON schema validation by throwing the error and exiting when it runs into an error. We need to modify this so that it is printing all the errors in the terminal like all the other efsity validating functions and only exiting on completion. This will make it easier to see all the errors and fix them once instead of having to run and fix the issues one at a time
The text was updated successfully, but these errors were encountered: