-
Notifications
You must be signed in to change notification settings - Fork 6
Added possibility to export the error Response of the parser #31
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
base: master
Are you sure you want to change the base?
Conversation
Sorry I let a own testfile in the PR, which i cleaned up now. (That is why the tests are failed) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks for the PR.
Minor comments.
errorResponse: | ||
description: 'Output the detailed response of the validation' | ||
default: "false" | ||
required: true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this should not be a required, but optional property.
As asked in #30
This PR added the possibility to log the error messages and details in the log of git action.