Skip to content

Commit f2a2039

Browse files
authored
[CI] Fix mypy check in CI (#1655)
### What this PR does / why we need it? Fix mypy check in CI: https://github.com/vllm-project/vllm-ascend/actions/runs/16115919385/job/45469646509?pr=1654 Mypy failed due to the greater numpy version. We need to pin `numpy=1.26.4` in vllm-ascend ### Does this PR introduce _any_ user-facing change? No ### How was this patch tested? CI passed Signed-off-by: MengqingCao <cmq0113@163.com>
1 parent 18495f4 commit f2a2039

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

.github/workflows/vllm_ascend_test.yaml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -103,10 +103,6 @@ jobs:
103103
pip install yapf==0.32.0
104104
yapf --diff --recursive .
105105
106-
- name: Install dependencies
107-
run: |
108-
pip install -r requirements-dev.txt --extra-index-url https://download.pytorch.org/whl/cpu
109-
110106
- name: Checkout vllm-project/vllm repo
111107
uses: actions/checkout@v4
112108
with:
@@ -126,6 +122,10 @@ jobs:
126122
pip install -r requirements/build.txt --extra-index-url https://download.pytorch.org/whl/cpu
127123
VLLM_TARGET_DEVICE=empty pip install .
128124
125+
- name: Install dependencies
126+
run: |
127+
pip install -r requirements-dev.txt --extra-index-url https://download.pytorch.org/whl/cpu
128+
129129
- name: Mypy Check
130130
run: |
131131
echo "::add-matcher::.github/workflows/matchers/mypy.json"

0 commit comments

Comments
 (0)