Skip to content
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

Make Yaml read exception handler more accurate. #15

Merged
merged 1 commit into from
Jun 19, 2020
Merged

Conversation

markpbaggett
Copy link
Owner

GitHub Issue: Issue 14

What Does this Do?

Addresses issue 14 by better handling YamlConstructionError exception.

How Should This Be Tested?

Build the branch and test running any operation versus a missing or extra field. You should see different error messages for both.

Additional Notes

The intent of this is to help people who may be using a new or old binary versus a different version of the associate config file.

@markpbaggett markpbaggett requested a review from mathewjordan May 25, 2020 18:03
@mathewjordan
Copy link
Collaborator

@markpbaggett This is good. Merging.

@mathewjordan mathewjordan merged commit 0e9e382 into master Jun 19, 2020
@markpbaggett markpbaggett deleted the Issue_14 branch March 3, 2021 13:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants