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

Drop support for EOL 3.5 #309

Merged
merged 5 commits into from
Jul 1, 2022
Merged

Drop support for EOL 3.5 #309

merged 5 commits into from
Jul 1, 2022

Conversation

hugovk
Copy link
Contributor

@hugovk hugovk commented Jun 12, 2021

Also:

@hugovk hugovk force-pushed the rm-3.5 branch 2 times, most recently from 2dda39a to 593ea89 Compare June 12, 2021 08:22
Copy link
Contributor

@mtreinish mtreinish left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code-wise this LGTM, but can you split this into 2 PRs, one adding 3.9 support to the package metadata (and probably most of the gha config cleanup) and the other dropping 3.5 support. I think for the next release (which should be relatively soon) we should still include 3.5 support and say in the release notes for it that 2.5.0 is the last release with 3.5 support. We haven't previously advertised dropping 3.5 before so I don't want to catch any potential 3.5 users by surprise.

@hugovk
Copy link
Contributor Author

hugovk commented Jun 12, 2021

Yep, that sounds reasonable, please see #310 for adding 3.9.

After that's merged (to avoid conflicts), I'll rebase and update this one. Thanks!

@hugovk hugovk changed the title Add support for Python 3.9, drop EOL 3.5 Drop support for EOL 3.5 Jun 26, 2021
@hugovk
Copy link
Contributor Author

hugovk commented Jun 26, 2021

I've created #313 to note 2.5.0 is the last release with Python 3.5 support.

PR #310 is merged and I've rebased this. The NEWS in this PR can be updated once 2.5.0 is out.

@hugovk

This comment has been minimized.

@hugovk
Copy link
Contributor Author

hugovk commented Nov 15, 2021

Merge conflict resolved!

Is there anything more needed here? Thanks!

@stephenfin
Copy link
Contributor

@hugovk Can you rebase this so we can pick up new test runs?

@hugovk
Copy link
Contributor Author

hugovk commented Feb 7, 2022

Rebased!

This needs the 3.5 status check being marked as NOT required.

At the same time, the new 3.10 one can be marked as required.

image

@hugovk
Copy link
Contributor Author

hugovk commented Jun 7, 2022

Updated to add a final, noop job to indicate success:

image

Its success depends on the build step fully passing. So instead of needing to #309 (comment) each and every time a Python version is added to or removed from the matrix, we can just add this new "test successful" as the required check.

@hugovk hugovk requested a review from mtreinish June 13, 2022 10:13
@hugovk
Copy link
Contributor Author

hugovk commented Jul 1, 2022

Closing this year-old PR to clear my list, happy to re-open if you want to merge :)

@hugovk hugovk closed this Jul 1, 2022
@jelmer
Copy link
Member

jelmer commented Jul 1, 2022

Sorry about that. If you reopen, I promise I'll take a look at merging it.

@hugovk
Copy link
Contributor Author

hugovk commented Jul 1, 2022

No problem, re-opened!

@hugovk hugovk reopened this Jul 1, 2022
@jelmer
Copy link
Member

jelmer commented Jul 1, 2022

Finally figured out how to mark the 3.5 tests as no longer required :)

@jelmer jelmer merged commit 621553d into testing-cabal:master Jul 1, 2022
@hugovk hugovk deleted the rm-3.5 branch July 1, 2022 18:54
@stephenfin
Copy link
Contributor

We probably want to do 3.6 next. I think that's EOL now too

@hugovk
Copy link
Contributor Author

hugovk commented Jul 2, 2022

Yes, 3.6 was EOL in December 2021 (3.7 will be June 2023).

https://endoflife.date/python

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.

4 participants