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
Uh oh!
There was an error while loading. Please reload this page.
-
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