Skip to content
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

Establish minimum Python version 3.6 for maint-1.9 branch #1079

Merged
merged 7 commits into from
Apr 4, 2022

Conversation

mwtoews
Copy link
Member

@mwtoews mwtoews commented Mar 8, 2022

This PR increases the minimum Python version to 3.6 for the maint-1.9 branch, for the upcoming Fiona 1.9.x release series.

It is mainly based on a few cherry picks from:

A few manual edits were required. It should have minimal conflicts with master. Compatibility with the Fiona 1.8.x series should be maintained.

@mwtoews mwtoews force-pushed the maint-1.9-minimum-python-3.6 branch from 8f3cb79 to 9b6741b Compare March 8, 2022 23:10
@mwtoews mwtoews requested a review from sgillies March 8, 2022 23:22
@mwtoews mwtoews added this to the 1.9 milestone Mar 8, 2022
@mwtoews
Copy link
Member Author

mwtoews commented Mar 23, 2022

Any glaring issues here @sgillies ? I might merge this sooner than later before it gets stale.

@mwtoews mwtoews merged commit e3bbb6b into Toblerity:maint-1.9 Apr 4, 2022
@mwtoews mwtoews deleted the maint-1.9-minimum-python-3.6 branch April 4, 2022 10:34
@sgillies
Copy link
Member

I'm going to remove 3.6 from the classifiers in setup.py. 3.7 will be the minimum Python version and we may yet increase to 3.8 before 1.9.0 as Python 3.11 is close.

@mwtoews
Copy link
Member Author

mwtoews commented Apr 27, 2022

@sgillies are you doing this? Otherwise, I can increase it to Python 3.8. There are other simple upgrades for Python 3.7 that I can see that I could slip in. Update: never-mind, I see the relevant commits now.

@mwtoews mwtoews mentioned this pull request Apr 28, 2022
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.

None yet

2 participants