Skip to content

Conversation

@jl-wynen
Copy link
Member

@jl-wynen jl-wynen commented May 7, 2024

This only changes the version constraints / selections. I will open a separate PR for pyupgrade (ruff check --select=UP) to keep this one here easier to review.

matrix:
os: [ubuntu-22.04]
build: [cp39, cp310]
build: [cp310]
Copy link
Member

Choose a reason for hiding this comment

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

Can we not build on 3.11?

Copy link
Member Author

Choose a reason for hiding this comment

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

We only build the oldest version here for tests of the other packages. We had both 3.9 and 3.10 to support the migration period which is now (hopefully) ending.

Copy link
Member

Choose a reason for hiding this comment

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

Wouldn't hurt to add 3.11 I suppose? Maybe we can drop 3.10 early in projects that do not need Mantid?

Copy link
Member Author

Choose a reason for hiding this comment

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

Could do. But I'd do that separately. It already takes a long time to test this PR.

@jl-wynen
Copy link
Member Author

jl-wynen commented May 7, 2024

All green. I'll update the branch protection rules when the PR is approved.

@jl-wynen jl-wynen merged commit 413b7a1 into main May 7, 2024
@jl-wynen jl-wynen deleted the drop-py39 branch May 7, 2024 09:42
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