-
Notifications
You must be signed in to change notification settings - Fork 31
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
feat: add python 3.11 support #142
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #142 +/- ##
=======================================
Coverage 99.78% 99.78%
=======================================
Files 11 11
Lines 461 461
Branches 33 33
=======================================
Hits 460 460
Partials 1 1 ☔ View full report in Codecov by Sentry. |
3b09999
to
c1cb129
Compare
tox.ini
Outdated
|
||
[pytest] | ||
addopts = --cov=tests --cov-report term --cov-config=.coveragerc -p no:randomly | ||
testpaths = tests | ||
|
||
[testenv] | ||
download = true |
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.
Please add a comment why this flag is needed?
requirements/common_constraints.txt
Outdated
# In edx-lint, until the pyjwt constraint in edx-lint has been removed. | ||
# See BOM-2721 for more details. | ||
# Below is the copied and edited version of common_constraints | ||
|
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.
It seems like an issue with the common_constraints.txt file as it haven't been updated and latest constraints have not been fetched properly. Perhaps try running make upgrade
again.
For the details, check the issue openedx/public-engineering#233
With python3.12 it has some failures so for this PR 3.11 is enough.