-
-
Notifications
You must be signed in to change notification settings - Fork 10.6k
[CI/Build] Adopt Mergify for auto-labeling PRs #9259
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
Conversation
👋 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:
🚀 |
Import initial configuration for [Mergify](https://docs.mergify.com/). This configuration does the following: - Auto-label PRs that change docs (`docs/` directory or any file ending in `.md`) with the `documentation` label - Auto-label PRs that go into conflict with `main` and ping the PR author to highlight that a conflict exists. Auto-remove this label when conflicts are resolved. This pattern serves as a starting point. I'm sure there are a lot more desired labeling additions. Please send me requests and I'll add them! Closes vllm-project#9192 Signed-off-by: Russell Bryant <[email protected]>
Signed-off-by: Russell Bryant <[email protected]>
3faedb8
to
d277439
Compare
Apply the `frontend` label when files under `vllm/entrypoints` change. Signed-off-by: Russell Bryant <[email protected]>
Signed-off-by: Russell Bryant <[email protected]> Signed-off-by: qishuai <[email protected]>
Signed-off-by: Russell Bryant <[email protected]> Signed-off-by: Randall Smith <[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: LeiWang1999 <[email protected]>
Import initial configuration for Mergify.
This configuration does the following:
Auto-label PRs that change docs (
docs/
directory or any fileending in
.md
) with thedocumentation
labelAuto-label PRs that change CI/Build related files with the
ci/build
label.Auto-label PRs with the
frontend
label if they change files undervllm/entrypoints
.Auto-label PRs with
kernel
if they change files undercsrc
. I know morepaths are needed here.
Auto-label PRs that go into conflict with
main
and ping the PRauthor to highlight that a conflict exists. Auto-remove this label
when conflicts are resolved.
This pattern serves as a starting point. I'm sure there are a lot more
desired labeling additions. Please send me requests and I'll add them!
Closes #9192
Signed-off-by: Russell Bryant [email protected]
cc @simon-mo
tasks that someone with commit access needs to do:
Add the Mergify github application to the
vllm-project
organization and thevllm
repository. See https://docs.mergify.com/getting-started/Create the following labels. The config uses them and I'm not positive if they
will get automatically created if they don't exist, so might as well just
create them.
needs-rebase
ci/build
frontend
kernel