Skip to content

Drop support for older Python versions - #25

Merged
diazona merged 4 commits into
masterfrom
drop-py2-support/1/dev
Dec 16, 2021
Merged

Drop support for older Python versions#25
diazona merged 4 commits into
masterfrom
drop-py2-support/1/dev

Conversation

@diazona

@diazona diazona commented Nov 30, 2021

Copy link
Copy Markdown
Contributor

This builds on #24 to drop support for Python 2, when we're ready for that.

@diazona diazona added this to the 0.6.0 milestone Nov 30, 2021
@diazona
diazona requested review from redtoad and removed request for redtoad November 30, 2021 23:32
@diazona diazona mentioned this pull request Dec 2, 2021
@diazona

diazona commented Dec 11, 2021

Copy link
Copy Markdown
Contributor Author

I pushed fixup commits to drop support for 3.3 as well, since I use enums in #21 and they were only added in 3.4. (And I'm kind of hesitant about supporting versions that aren't tested by the continuous integration script anyway) I figured we might as well do that at the same time as dropping Python 2 support.

Happy to abandon those commits and keep support for 3.3 if anyone wants to argue for it, though.

@diazona
diazona requested review from coordt and redtoad December 11, 2021 23:00
Comment thread setup.py Outdated
Comment thread setup.py Outdated
Comment thread .github/workflows/tests.yml
@diazona
diazona force-pushed the drop-py2-support/1/dev branch from c0b0e23 to ea8f9e1 Compare December 12, 2021 22:01
@diazona diazona changed the title Drop Python 2 support Drop support for older Python versions Dec 12, 2021
@diazona
diazona force-pushed the drop-py2-support/1/dev branch from ea8f9e1 to 27dafd8 Compare December 13, 2021 20:44
@diazona
diazona marked this pull request as ready for review December 13, 2021 20:44
This commit removes the tox environments for Python 2.7, 3.3, and 3.4,
since we no longer plan to officially support those versions.
This commit edits setup.py to require Python 3.5 or greater, and removes
the trove classifiers for Pythons 2, 2.7, 3.3, and 3.4, while adding
the classifier for Python 3 only.
@diazona
diazona force-pushed the drop-py2-support/1/dev branch from 27dafd8 to d459d45 Compare December 13, 2021 20:50
@diazona
diazona requested a review from coordt December 13, 2021 20:50
@diazona

diazona commented Dec 13, 2021

Copy link
Copy Markdown
Contributor Author

@coordt you mind giving a final sanity check?

@diazona
diazona merged commit c09cbbc into master Dec 16, 2021
@diazona
diazona deleted the drop-py2-support/1/dev branch December 16, 2021 19:23
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.

2 participants