-
Notifications
You must be signed in to change notification settings - Fork 10
Drop EOL Pythons #35
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 EOL Pythons #35
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.
Nice, there still a few require changes and some nice to haves for this PR:
- update setup.py, python_requires, drop out of date classifiers, remove import multiprocessing workaround, remove ez_setup imports.
- remove ez_setup.py
- update build-wheels.sh script to skip python <3.6 similar to what we do for pymongo: https://github.com/mongodb/mongo-python-driver/blob/a94916edf10fddad3335ff14b55b83e27641ec14/.evergreen/build-manylinux-internal.sh#L14
- Remove PY_MAJOR_VERSION code from src/bsonjs.c.
- Remove sys.version_info specific code from the test suite.
…build-wheels.sh to skip older python versions
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, thanks!
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!
No description provided.