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.

Sign up for GitHub

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

Jump to bottom

[TODO] Run MyPy against multiple Pythons #2144

Open
webknjaz opened this issue Dec 12, 2024 · 0 comments
Open

[TODO] Run MyPy against multiple Pythons #2144

webknjaz opened this issue Dec 12, 2024 · 0 comments
Assignees
@webknjaz
Labels
enhancement Improvements to functionality maintenance Related to maintenance processes PR wanted Feature is discussed or bug is confirmed, PR needed refactor Refactoring code tests Testing and related things

Comments

@webknjaz
Copy link
Member

The current MyPy invocation is happening under the supposedly lowest supported version Python. But it may behave differently if checked against newer versions. I typically implement testing against every other version or lowest+highest at least.
The actual invocation runtime can be newer as there's a CLI flag for controlling this.
@webknjaz webknjaz added PR wanted Feature is discussed or bug is confirmed, PR needed refactor Refactoring code tests Testing and related things maintenance Related to maintenance processes enhancement Improvements to functionality labels Dec 12, 2024
@webknjaz webknjaz self-assigned this Dec 12, 2024
@webknjaz webknjaz moved this to 🤔 Brain 🧠 dump 💡 in 📅 Procrastinating in public Dec 12, 2024
webknjaz added a commit to Mr-Sunglasses/pip-tools that referenced this issue Dec 12, 2024
@Mr-Sunglasses @webknjaz
🧪 Bump MyPy Python to 3.9 @ pre-commit config
94b0149 
The `pre-commit.ci` service dropped support for Python 3.8 as it's
gone EOL. This change makes the check runnable again. It is suboptimal
as we haven't yet dropped support for Python 3.8. jazzband#2144 will address
the underlying issue more thoroughly.

Resolves jazzband#2133.

Co-Authored-By: Sviatoslav Sydorenko <[email protected]>
@webknjaz webknjaz mentioned this issue Dec 12, 2024
Merged
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@webknjaz webknjaz

Labels
enhancement Improvements to functionality maintenance Related to maintenance processes PR wanted Feature is discussed or bug is confirmed, PR needed refactor Refactoring code tests Testing and related things
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@webknjaz