-
-
Notifications
You must be signed in to change notification settings - Fork 17
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
Remove compatibility with legacy versions of Python #64
Remove compatibility with legacy versions of Python #64
Conversation
cc89324
to
19846b2
Compare
ab9c1c2
to
daaaea4
Compare
Quality Gate passedIssues Measures |
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #64 +/- ##
==========================================
+ Coverage 80.99% 81.19% +0.20%
==========================================
Files 133 132 -1
Lines 29503 29299 -204
Branches 5066 5044 -22
==========================================
- Hits 23895 23789 -106
+ Misses 4735 4641 -94
+ Partials 873 869 -4 ☔ View full report in Codecov by Sentry. |
CodeFactor is merely running % % suspicious-pickle-usage (S301)Derived from the flake8-bandit linter. What it doesChecks for calls to Why is this bad?Deserializing untrusted data with Avoid deserializing untrusted data with If you must deserialize untrusted data with Exampleimport pickle
with open("foo.pickle", "rb") as file:
foo = pickle.load(file) Use instead: import json
with open("foo.json", "rb") as file:
foo = json.load(file) References |
When this is released, please reduce |
The list of supported Python versions on Pypi should be fixed when we do the next release. I will be doing a version increase to 3.0.0 after this is merged in an hour or so, since this PR and other changes we've done before will most likely bring breaking changes to projects using |
Description
Please include:
Closes: # (issue)
Remove compatibility with legacy versions of Python
Remove
src/whoosh/compat.py
and then use Python 3 syntax.Copy
b()
,memoryview_()
, andu()
into files that use them on variables to encourage future local refactoring.Checklist: