-
-
Notifications
You must be signed in to change notification settings - Fork 11.2k
Increase operation per run limit for "Close inactive issues and PRs" workflow #9661
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:
🚀 |
Signed-off-by: Harry Mellor <[email protected]>
Signed-off-by: Harry Mellor <[email protected]>
russellb
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm, thanks!
…workflow (vllm-project#9661) Signed-off-by: Harry Mellor <[email protected]> Signed-off-by: Erkin Sagiroglu <[email protected]>
…workflow (vllm-project#9661) Signed-off-by: Harry Mellor <[email protected]> Signed-off-by: qishuai <[email protected]>
…workflow (vllm-project#9661) Signed-off-by: Harry Mellor <[email protected]> Signed-off-by: Sumit Dubey <[email protected]>
…workflow (vllm-project#9661) Signed-off-by: Harry Mellor <[email protected]> Signed-off-by: LeiWang1999 <[email protected]>
Increases
operations-per-runfrom30to1000.This ensures that changes to this workflow propagate to all issues and PRs in days rather than months.
In the most recent run (https://github.com/vllm-project/vllm/actions/runs/11491223727/job/31983238963#step:2:19447) you can see that there are ~5k operations left. So increasing the limit to 1000 leaves 4000 for other tasks. The usage will only get close to 1000 when the workflow file is changed (or new, as it is now).