-
-
Notifications
You must be signed in to change notification settings - Fork 5.1k
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/Build] Improve mypy + python version matrix #10041
Conversation
👋 Hi! Thank you for contributing to the vLLM project. Once the PR is approved and ready to go, your PR reviewer(s) can run CI to test the changes comprehensively before merging. To run CI, PR reviewers can do one of these:
🚀 |
This pull request has merge conflicts that must be resolved before it can be |
f416483
to
e80e876
Compare
The `mypy` CI job ran using multiple Python versions, though it was always checking for compatibility with Python 3.9 because of the config option in `pyproject.toml`. This change makes it so local usage of `format.sh` will still default to checking for compatibility with the lowest Python version (3.9). The CI job will now check for compatibility with each python version used in the matrix instead of only running `mypy` itself with different versions, but always checking 3.9 compatibility. Signed-off-by: Russell Bryant <[email protected]>
e80e876
to
b773dc1
Compare
Signed-off-by: Russell Bryant <[email protected]> Signed-off-by: Isotr0py <[email protected]>
Signed-off-by: Russell Bryant <[email protected]> Signed-off-by: Loc Huynh <[email protected]>
Signed-off-by: Russell Bryant <[email protected]> Signed-off-by: Sumit Dubey <[email protected]>
Signed-off-by: Russell Bryant <[email protected]>
Signed-off-by: Russell Bryant <[email protected]> Signed-off-by: Maxime Fournioux <[email protected]>
Signed-off-by: Russell Bryant <[email protected]> Signed-off-by: Tyler Michael Smith <[email protected]>
Signed-off-by: Russell Bryant <[email protected]>
b773dc1 [CI/Build] Improve mypy + python version matrix
commit b773dc1
Author: Russell Bryant [email protected]
Date: Tue Nov 5 15:08:47 2024 +0000