-
Notifications
You must be signed in to change notification settings - Fork 617
bump default python version to 3.11 #2072
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
bump default python version to 3.11 #2072
Conversation
Yikun
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Otherwise LGTM
.pre-commit-config.yaml
Outdated
| - id: mypy-3.10 # TODO: Use https://github.com/pre-commit/mirrors-mypy when mypy setup is less awkward | ||
| name: Run mypy for Python 3.10 | ||
| entry: tools/mypy.sh 1 "3.10" | ||
| - id: mypy-3.11 # TODO: Use https://github.com/pre-commit/mirrors-mypy when mypy setup is less awkward |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This shouldn't be changed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should I keep the original code like this:
- id: mypy-3.10 # TODO: Use https://github.com/pre-commit/mirrors-mypy when mypy setup is less awkward
name: Run mypy for Python 3.10
entry: tools/mypy.sh 1 "3.10"
Or just remove the # TODO comment and just keep:
- id: mypy-3.11
name: Run mypy for Python 3.11
entry: tools/mypy.sh 1 "3.11"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
keep the original code, this PR only make 3.11 as default, but not drop 3.10 support
Signed-off-by: ChenTaoyu-SJTU <[email protected]>
b169533 to
6329192
Compare
### What this PR does / why we need it? Bump default python version to 3.11, see vllm-project#1980 ### Does this PR introduce _any_ user-facing change? no ### How was this patch tested? pass CI - vLLM version: v0.10.0 - vLLM main: vllm-project/vllm@12a223e Signed-off-by: ChenTaoyu-SJTU <[email protected]>
### What this PR does / why we need it? Bump default python version to 3.11, see vllm-project#1980 ### Does this PR introduce _any_ user-facing change? no ### How was this patch tested? pass CI - vLLM version: v0.10.0 - vLLM main: vllm-project/vllm@12a223e Signed-off-by: ChenTaoyu-SJTU <[email protected]> Signed-off-by: weijinqian_v1 <[email protected]>
### What this PR does / why we need it? Upgrade python to 3.11 Soft backport: #2072 ### Does this PR introduce _any_ user-facing change? No ### How was this patch tested? CI passed Signed-off-by: Yikun Jiang <[email protected]>
### What this PR does / why we need it? Switch Infra to linux-aarch64-a2 and python to 3.11 Soft backport: #2065 Soft backport: #2072 ### Does this PR introduce _any_ user-facing change? No ### How was this patch tested? CI passed search all: `linux-arm64-npu` and `3.10` --------- Signed-off-by: Yikun Jiang <[email protected]>
### What this PR does / why we need it? Bump default python version to 3.11, see vllm-project#1980 ### Does this PR introduce _any_ user-facing change? no ### How was this patch tested? pass CI - vLLM version: v0.10.0 - vLLM main: vllm-project/vllm@12a223e Signed-off-by: ChenTaoyu-SJTU <[email protected]>
### What this PR does / why we need it? Bump default python version to 3.11, see vllm-project#1980 ### Does this PR introduce _any_ user-facing change? no ### How was this patch tested? pass CI - vLLM version: v0.10.0 - vLLM main: vllm-project/vllm@12a223e Signed-off-by: ChenTaoyu-SJTU <[email protected]>
What this PR does / why we need it?
Bump default python version to 3.11, see #1980
Does this PR introduce any user-facing change?
no
How was this patch tested?
pass CI