Skip to content

Increase Python version checked my mypy in CI - #123

Closed
luleyleo wants to merge 3 commits into
python-sdbus:masterfrom
luleyleo:fix-ci
Closed

Increase Python version checked my mypy in CI#123
luleyleo wants to merge 3 commits into
python-sdbus:masterfrom
luleyleo:fix-ci

Conversation

@luleyleo

Copy link
Copy Markdown
Contributor

mypy no longer supports version 3.9, incrementing it to 3.10 should fix the CI.

mypy no longer supports version 3.9
@igo95862

Copy link
Copy Markdown
Collaborator

My plan is to switch CI to use the Debian 12 which ships mypy that can check for Python 3.9.

The next major version will probably increase the Python version to 3.10 or 3.11 anyway.

@luleyleo

Copy link
Copy Markdown
Contributor Author

Oh, I see. If you want to stick with mypy 1.9, why not just use ubuntu-24.04 as the lint runner? It seems to have the right version: https://packages.ubuntu.com/noble/mypy

@luleyleo

Copy link
Copy Markdown
Contributor Author

Never mind, apparently it somehow doesn't...

@igo95862

Copy link
Copy Markdown
Collaborator

Because the mypy is installed from PyPI. I am working on updating the CI including the checkout action versions which are no longer supported.

@igo95862

Copy link
Copy Markdown
Collaborator

@luleyleo

Copy link
Copy Markdown
Contributor Author

I think I've found a nice way to pin mypy to it last pre-2.0 version

@luleyleo

Copy link
Copy Markdown
Contributor Author

Maybe that approach works well for your CI update branch?

@igo95862

Copy link
Copy Markdown
Collaborator

CI is passing in my branch: 5a58c17

@igo95862

Copy link
Copy Markdown
Collaborator

I will merge it and create a new 0.14.3 release.

@luleyleo

Copy link
Copy Markdown
Contributor Author

Alright, thank you! I'll close this one then

@luleyleo luleyleo closed this Aug 20, 2026
@igo95862

Copy link
Copy Markdown
Collaborator

0.14.3 released

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