-
-
Notifications
You must be signed in to change notification settings - Fork 68
Drop Python 2.7 and 3.5 support and bump dependency upper bounds #137
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
Conversation
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.
LGTM … whatever you people make out of that 🙈
@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 |
.github/workflows/python-test.yml
Outdated
@@ -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] |
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.
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.
c9e3ed8
to
91a75b2
Compare
Codecov Report
@@ 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
Continue to review full report at Codecov.
|
@RobbeSneyders thank you for the contribution |
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 ofjsonschema
. Both upstream dependencies and downstream dependents have already upgraded theirjsonschema
version constraints, which leads to conflicts withopenapi-spec-validator
.