Skip to content

Allow ValueError to pass-through without exception conversion. #44

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

EliRibble
Copy link

This maintains compatibility with the Python json library which
raises ValueError when it cannot parse a file.

This maintains compatibility with the Python json library which
raises ValueError when it cannot parse a file.
@EliRibble
Copy link
Author

This is to address #43

@vaidik
Copy link
Owner

vaidik commented Aug 23, 2020

Thanks for the change. I see your point. Which version of Python are you using? I tried testing against Python 3.8 and I am getting json.decoder.JSONDecodeError instead.

Also, some tests are failing. Can you check these once please?

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