-
-
Notifications
You must be signed in to change notification settings - Fork 5.3k
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] Always run mypy #10122
[CI/Build] Always run mypy #10122
Conversation
If you would like to configure the repo to block auto-merges if the mypy workflow fails, you will first need to merge this PR so that the workflow runs on all PRs, regardless of the files that have changed. This is because github configuration does not support blocking on workflows that run conditionally. Signed-off-by: Russell Bryant <[email protected]>
👋 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:
🚀 |
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]>
If you would like to configure the repo to block auto-merges if the
mypy workflow fails, you will first need to merge this PR so that the
workflow runs on all PRs, regardless of the files that have changed.
This is because github configuration does not support blocking on
workflows that run conditionally.
Signed-off-by: Russell Bryant [email protected]