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

Drop Python 2.7 and 3.5 support and bump dependency upper bounds #137

Merged
merged 3 commits into from
Jan 27, 2022

Conversation

RobbeSneyders
Copy link
Contributor

Fixes #130
Fixes #135

jsonschema released version 4.0.0 which dropped support for Python 2.7 and 3.5. openapi-spec-validator should do the same to support new versions of jsonschema. Both upstream dependencies and downstream dependents have already upgraded their jsonschema version constraints, which leads to conflicts with openapi-spec-validator.

Copy link

@sthagen sthagen left a comment

Choose a reason for hiding this comment

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

LGTM … whatever you people make out of that 🙈

pyproject.toml Outdated Show resolved Hide resolved
pyproject.toml Outdated Show resolved Hide resolved
@nezhar
Copy link
Contributor

nezhar commented Jan 21, 2022

@p1c2u I would also suggest to drop support for Python 3.6 and to add suport for Python 3.10 acording to https://endoflife.date/python

@RobbeSneyders
Copy link
Contributor Author

Good catch @p1c2u, addressed your comments.
I also dropped python 3.6 in favor of 3.10 as mentioned by @nezhar. This is in line with jsonschema as well.

@@ -14,7 +14,7 @@ jobs:
runs-on: ${{ matrix.os }}
strategy:
matrix:
python-version: [2.7, 3.5, 3.6, 3.7, 3.8, 3.9]
python-version: [3.7, 3.8, 3.9, 3.10]
Copy link
Contributor

Choose a reason for hiding this comment

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

The Python versions should be declared as strings as in the docs of the setup python action, otherwise 3.10 might be interpreted as 3.1 in the workflow.

@p1c2u p1c2u force-pushed the master branch 3 times, most recently from c9e3ed8 to 91a75b2 Compare January 27, 2022 14:24
@codecov
Copy link

codecov bot commented Jan 27, 2022

Codecov Report

Merging #137 (04f3017) into master (f8645de) will decrease coverage by 1.89%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #137      +/-   ##
==========================================
- Coverage   98.14%   96.24%   -1.90%     
==========================================
  Files          19       19              
  Lines         538      533       -5     
  Branches        0      101     +101     
==========================================
- Hits          528      513      -15     
  Misses         10       10              
- Partials        0       10      +10     
Impacted Files Coverage Δ
openapi_spec_validator/__init__.py 100.00% <100.00%> (ø)
openapi_spec_validator/generators.py 100.00% <100.00%> (ø)
openapi_spec_validator/handlers/file.py 100.00% <100.00%> (ø)
openapi_spec_validator/handlers/requests.py 100.00% <100.00%> (ø)
openapi_spec_validator/handlers/urllib.py 100.00% <100.00%> (ø)
openapi_spec_validator/handlers/utils.py 100.00% <100.00%> (ø)
openapi_spec_validator/readers.py 100.00% <100.00%> (ø)
openapi_spec_validator/schemas.py 100.00% <100.00%> (ø)
openapi_spec_validator/shortcuts.py 100.00% <100.00%> (ø)
openapi_spec_validator/validators.py 97.22% <100.00%> (-1.86%) ⬇️
... and 4 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update eb169e7...04f3017. Read the comment docs.

@p1c2u p1c2u merged commit 13f27f2 into python-openapi:master Jan 27, 2022
@p1c2u
Copy link
Collaborator

p1c2u commented Jan 27, 2022

@RobbeSneyders thank you for the contribution

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.

Incompatible with openapi-schema-validator 0.2.0 openapi_spec_validator fails with jsonschema 4.0
4 participants