Ci/unassign inactive assignees - #2654
Conversation
…fecycle management Signed-off-by: rohitjyoti <73464863+rohitjyoti@users.noreply.github.com>
…omation Signed-off-by: rohitjyoti <73464863+rohitjyoti@users.noreply.github.com>
Replace malformed heredoc with standard multi-line string to resolve invalid workflow file error on line 54. Signed-off-by: rohitjyoti <73464863+rohitjyoti@users.noreply.github.com>
Use separate -m flags for multi-line commit message instead of unquoted newline. Signed-off-by: rohitjyoti <73464863+rohitjyoti@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…ctive-assignees feature - Revert .github/workflows/contributor-leaderboard.yml to main - Revert .github/workflows/owner-notification.yml to main - Revert .github/workflows/test-and-build.yml to main - Keep unassign-inactive-assignees.yml from feature branch This ensures all other changes from main are applied while preserving the primary feature. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
✅ Deploy Preview for vllm-semantic-router ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
github-actions
Bot
assigned
AayushSaini101,
drivebyer,
FAUST-BENCHOU,
ramkrishs,
rootfs,
shraderdm,
WUKUNTAI-0211 and
Xunzhuo
👥 vLLM Semantic Team NotificationThe following members have been identified for the changed files in this PR and have been automatically assigned when their GitHub accounts are assignable in this repository: 📁
|
✅ Supply Chain Security Report — All Clear
Scanned at |
Xunzhuo
left a comment
Xunzhuo
left a comment
There was a problem hiding this comment.
Blocking correctness issues: 1) the warning comment and existing stale workflow update issue.updated_at, so bot writes reset the inactivity clock and can prevent the intended 30-day unassignment; compute activity from human events instead. 2) linked-PR detection scans only the first 100 PRs, uses body substring matching, and fails open on API errors, which can remove an assignee despite active work; paginate and fail closed. 3) if unassignment succeeds but notification fails, the next run skips the issue permanently; make the operation recoverable. 4) remove unrelated changes to test-and-build.yml, contributor-leaderboard.yml, and owner-notification.yml, which disable merge-queue testing, restore direct main pushes, and regress the owner notification theme. The tracking labels also need to be provisioned rather than silently ignored.
9 participants
Closes #2586
Purpose
CI/BuildTest Plan
What commands, checks, or manual steps should reviewers use?
make agent-lint CHANGED_FILES=".github/workflows/unassign-inactive-assignees.yml,.github/workflows/test-and-build.yml"Why is this validation sufficient for the affected module(s)? CI/Build changes require workflow syntax validation and verification that scheduled jobs execute correctly. The logic is self-evident through logs when the workflow runs.
Test Result
What were the actual results?
packages:writepermission added to test-and-build for nested docker-publish jobAny follow-up risks, gaps, or blockers? None. Workflow runs safely with read-only operations until assignment removal. All errors are logged for visibility.