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

Be able to get the data on error #25

Merged
merged 1 commit into from
Jun 4, 2021
Merged

Be able to get the data on error #25

merged 1 commit into from
Jun 4, 2021

Conversation

sbrunner
Copy link
Owner

@sbrunner sbrunner commented Jun 3, 2021

No description provided.

@sbrunner sbrunner added the backport master Backport the pull request to the 'master' branch label Jun 3, 2021
@sbrunner sbrunner requested a review from ger-benjamin June 3, 2021 07:37
@@ -47,6 +47,7 @@ def main() -> None:
config = validate.load(args.config, json.loads(schema_data))
except validate.ValidationError as error:
LOG.error(error)
config = error.data

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

and ? you don't do anything with these data ? It's just to be able to debug ?

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I set the config that's used at line 54 and next, the skip_config_errors was just not working before...

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah yes, sorry

@sbrunner sbrunner merged commit 290b9c5 into 0.9 Jun 4, 2021
@sbrunner sbrunner deleted the data-error branch June 4, 2021 06:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport master Backport the pull request to the 'master' branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants