chore: retire Mergify-replaceable reusable workflows (T8937)#146
Conversation
Deletes 5 reusable workflows + labeler.yml now that the consumer fleet has been swept (Phase C + the Phase B straggler/LTS sweeps). Functionality preserved by central Mergify rules per spec §4a (base-branch labeling), §4b (author-assign), and the existing rules in vyos/mergify. Deleted: - .github/workflows/add-pr-labels.yml - .github/workflows/add-rebase-label.yml - .github/workflows/assign-author.yml - .github/workflows/check-pr-message.yml - .github/workflows/label-backport.yml - .github/labeler.yml (no remaining consumer) Preserved: - check-pr-conflict.yml — final probe found 2 live consumers (vyos/vyos-documentation current + heads/rolling); deletion deferred. - check-pr-merge-conflict.yml (T8934 committed-marker case Mergify cannot replicate). - check-stale.yml (PR + issue stale stays in GHA per spec §2 deferred). - All lint/scan/CLA/build/trigger workflows + get-token composite. Advances: T8937 🤖 Generated by [robots](https://vyos.io)
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository YAML (base), Central YAML (inherited), Organization UI (inherited) Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughSummary by CodeRabbit
WalkthroughThis PR removes repo-local PR triage automation: five reusable workflows and the branch-based labeler; AGENTS.md is updated to reference central Mergify rules (see edited lines in Files/lines cited: ChangesGitHub automation removal
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches✨ Simplify code
Comment |
…w findings) Advances: T8937 🤖 Generated by [robots](https://vyos.io)
Adversarial review —
|
Deletes 5 reusable workflows +
labeler.ymlfromvyos/.github. The consumer fleet has been swept — Phase C (73 in-scope repos, tiered), the 2026-06-03 VN LTS sweep (10 PRs), and the 2026-06-09/10 final straggler sweep (6 PRs:vyos/vyatta-bash@rolling+ 3 VN hotfix + 2 VN security branches, all merged). Final fail-closed probe across all branches of all 73 in-scope repos: zero non-exempt stragglers, zero probe failures (see Phorge T8937 for the full rollout + exemption record).Functionality is preserved by central Mergify rules in
vyos/mergify:Label PR with <branch>(toggle rules)Auto-assign PR authorinvalid-task-idrule + centralinvalid-title/invalid-bodyprotections (T8966)Preserved in this repo:
check-pr-conflict.yml— final probe found 2 live consumers (vyos/vyos-documentationcurrent+heads/rolling); deletion deferred to a follow-up after those callers are retired.check-pr-merge-conflict.yml(T8934 committed-marker case Mergify cannot replicate)check-stale.yml(PR + issue stale stays in GHA per spec §2)PR stale + issue stale stay in
check-stale.yml(PR-stale migration deferred per spec §2).The VyOS-Networks/.github twin PR follows; together they complete the rollout — central Mergify becomes the sole source of truth for fleet-wide PR automation.
Advances: T8937
🤖 Generated by robots