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

Fixed wrong example code for using the validator #518

Merged
merged 1 commit into from
Dec 12, 2022
Merged

Fixed wrong example code for using the validator #518

merged 1 commit into from
Dec 12, 2022

Conversation

patrickl92
Copy link
Contributor

Purpose / Goal

Fixes an error in the example code, where the boolean parameter to enable validation needs to be specified on the parse function, not on the constructor of the XMLParser.
I also moved the constructor out of the try/catch block, to easier indicate where the error will be thrown.

Type

  • [x]Bug Fix
  • [ ]Refactoring / Technology upgrade
  • [ ]New Feature

@coveralls
Copy link

coveralls commented Dec 3, 2022

Coverage Status

Coverage remained the same at 98.447% when pulling 3c9eaaf on patrickl92:master into f900ef8 on NaturalIntelligence:master.

@amitguptagwl amitguptagwl merged commit 0d9d60d into NaturalIntelligence:master Dec 12, 2022
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.

3 participants