-
Notifications
You must be signed in to change notification settings - Fork 71
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
Bug: jsonschema.compat #149
Comments
Given that this project supports 2.7 and jsonschema 4.0 drop support for 2.7 it makes sense to pin the version number in setup.py . |
See also python-jsonschema/jsonschema#845 |
so maybe #148 this the fix do - just pin it in the requirements? But this doesnt seem so nice a solution |
This seems to be an issue with python_requires being not honored. This is fixed in python-jsonschema/jsonschema@e61dcbc . There is also 4.0.1 tagged with the commit https://github.com/Julian/jsonschema/releases/tag/v4.0.1 . So probably once 4.0.0 is yanked with the fixed version (4.0.1) in PyPI tools will continue to pull jsonschema < 4.0.0 automatically. |
Sorry, I was wrong here. It seems the version for jsonschema needs to be pinned here or to fix jsonschema.compat usage netbox-community/netbox#7401 |
Yep, swagger_spec_validator should be fixed to work with jsonschema==4.x or set in setup.py max version 3.x temporary fix is to install manually version 3.x |
#146 this is a good solution - using 'six' |
What is the status of this issue? |
I think it go fixed in 734c7e1#diff-f63c3ee26a4134e668551fb48835df29afe314537b277b8ffe33de979262210b So I reckon this issue can be closed |
ModuleNotFoundError: No module named 'jsonschema.compat
Jsonschema has a new version out, where this does not exist, and causes an error.
https://github.com/Yelp/swagger_spec_validator/blob/master/swagger_spec_validator/ref_validators.py#L14
See error more detail here
The text was updated successfully, but these errors were encountered: