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

Ci - drop 37 support #966

Merged
merged 5 commits into from Oct 25, 2022
Merged

Conversation

xmatthias
Copy link
Contributor

@xmatthias xmatthias commented Oct 25, 2022

Drop python 37 wheels
as mentioned in #962 (comment) - there's only really a point in having wheels for 3.8, 3.9, 3.10 and 3.11 (supported versions of python).

Based on this, this PR will aim to

  • Remove CI for 3.6 and 3.7
  • Remove wheel building for 3.6 and 3.7
  • Update the pypi classifier to match the supported versions
  • Update CI actions to avoid action deprecation warnings

About this last point, only conda-incubator/setup-miniconda is still left - due to the issue linked above (they didn't release an updated new version so far).

The resulting artifact.zip file will contain the following files (notice that 3.6 and 3.7 are now gone).

screenshot

A "finished" wheels run can be found here - which is the run from the last commit (wheel CI doesn't run on PR's, so you'd not see the result otherwise).

@xmatthias xmatthias changed the title Ci/drop 37 support Ci - drop 37 support Oct 25, 2022
@@ -196,7 +192,7 @@ jobs:
fail-fast: false
matrix:
os: [ 'ubuntu-latest', 'windows-latest', 'macos-latest' ]
python-version: ['3.7', '3.8', '3.9','3.10', '3.11-dev']
python-version: ['3.8', '3.9','3.10', '3.11-dev']
Copy link
Member

Choose a reason for hiding this comment

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

Any idea on when we could use plain '3.11' instead of '3.11-dev' ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

not really - the way i understood this, the "final" version of python needs to be in https://github.com/actions/python-versions/blob/main/versions-manifest.json.
considering 3.11 stable has only been released yesterday, i assume it'll still take a few days - but then changing to "3.11" is absolutely the right approach!

Copy link
Member

@FrancescAlted FrancescAlted left a comment

Choose a reason for hiding this comment

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

This looks good to me. I just have a minor comment.

Copy link
Member

@avalentino avalentino left a comment

Choose a reason for hiding this comment

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

LGTM

@FrancescAlted FrancescAlted merged commit e82803d into PyTables:master Oct 25, 2022
@xmatthias xmatthias deleted the ci/drop_37_support branch October 25, 2022 16:07
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

3 participants