Replies: 1 comment
-
Resolved:
|
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
Someone please check pre-commit for
https://github.com/vllm-project/vllm/actions/runs/13255196956/job/37000775151?pr=7086
We tried to add
# noqa: E712
to bypass this ruff check but didn't work. Not sure why.We must check for
False
via== False
as that line of code has 3 states,None
,False
, andOther
. None and False are not equivalent. Please remove this rule or let us bypass it.Beta Was this translation helpful? Give feedback.
All reactions