From 73b862bb1f7367e0c23a7448cb7a8140e2674e5d Mon Sep 17 00:00:00 2001 From: jordanthejet Date: Fri, 6 Mar 2026 17:20:44 -0500 Subject: [PATCH] fix: address round-2 review comments on PR #2928 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - docs/pr-workflow.md: replace hardcoded maintainer handles with generic WORKFLOW_OWNER_LOGINS + CODEOWNERS reference - docs/vi/pr-workflow.md, docs/i18n/vi/pr-workflow.md: fix awkward "và hoặc là" phrasing, sync new branch-model bullets (workflow-owner config line + all PRs target master directly) Co-Authored-By: Claude Opus 4.6 --- docs/i18n/vi/pr-workflow.md | 4 +++- docs/pr-workflow.md | 2 +- docs/vi/pr-workflow.md | 4 +++- 3 files changed, 7 insertions(+), 3 deletions(-) diff --git a/docs/i18n/vi/pr-workflow.md b/docs/i18n/vi/pr-workflow.md index 3c3b0662d..1877704ce 100644 --- a/docs/i18n/vi/pr-workflow.md +++ b/docs/i18n/vi/pr-workflow.md @@ -100,8 +100,10 @@ Duy trì các quy tắc branch protection sau trên `master`: - Yêu cầu review pull request trước khi merge. - Yêu cầu review CODEOWNERS cho các đường dẫn được bảo vệ. - Với `.github/workflows/**`, yêu cầu phê duyệt từ owner qua `CI Required Gate` (`WORKFLOW_OWNER_LOGINS`) và giới hạn quyền bypass branch/ruleset cho org owner. +- Danh sách workflow-owner mặc định được cấu hình qua biến repository `WORKFLOW_OWNER_LOGINS` (xem CODEOWNERS cho maintainer hiện tại). - Hủy bỏ approval cũ khi có commit mới được đẩy lên. - Hạn chế force-push trên các branch được bảo vệ. +- Tất cả PR của contributor nhắm trực tiếp vào `master`. --- @@ -213,7 +215,7 @@ Chúng tôi **không** yêu cầu contributor định lượng quyền sở hữ - Mục tiêu triage maintainer đầu tiên: trong vòng 48 giờ. - Nếu PR bị chặn, maintainer để lại một checklist hành động được. - Tự động hóa `stale` được dùng để giữ hàng đợi lành mạnh; maintainer có thể áp dụng `no-stale` khi cần. -- Tự động hóa `pr-hygiene` kiểm tra các PR mở mỗi 12 giờ và đăng nhắc nhở khi PR không có commit mới trong 48+ giờ và hoặc là đang tụt hậu so với `master` hoặc thiếu/thất bại `CI Required Gate` trên head commit. +- Tự động hóa `pr-hygiene` kiểm tra các PR mở mỗi 12 giờ và đăng nhắc nhở khi PR không có commit mới trong 48+ giờ và rơi vào một trong hai trường hợp: đang tụt hậu so với `master` hoặc thiếu/thất bại `CI Required Gate` trên head commit. ### 8.1 Kiểm soát ngân sách hàng đợi diff --git a/docs/pr-workflow.md b/docs/pr-workflow.md index e90865615..0571929e7 100644 --- a/docs/pr-workflow.md +++ b/docs/pr-workflow.md @@ -100,7 +100,7 @@ Maintain these branch protection rules on `master`: - Require pull request reviews before merge. - Require CODEOWNERS review for protected paths. - For `.github/workflows/**`, require owner approval via `CI Required Gate` (`WORKFLOW_OWNER_LOGINS`) and keep branch/ruleset bypass limited to org owners. -- Default workflow-owner allowlist includes `theonlyhennygod` and `jordanthejet` (plus any comma-separated additions from `WORKFLOW_OWNER_LOGINS`). +- Default workflow-owner allowlist is configured via the `WORKFLOW_OWNER_LOGINS` repository variable (see CODEOWNERS for current maintainers). - Dismiss stale approvals when new commits are pushed. - Restrict force-push on protected branches. - All contributor PRs target `master` directly. diff --git a/docs/vi/pr-workflow.md b/docs/vi/pr-workflow.md index 3c3b0662d..1877704ce 100644 --- a/docs/vi/pr-workflow.md +++ b/docs/vi/pr-workflow.md @@ -100,8 +100,10 @@ Duy trì các quy tắc branch protection sau trên `master`: - Yêu cầu review pull request trước khi merge. - Yêu cầu review CODEOWNERS cho các đường dẫn được bảo vệ. - Với `.github/workflows/**`, yêu cầu phê duyệt từ owner qua `CI Required Gate` (`WORKFLOW_OWNER_LOGINS`) và giới hạn quyền bypass branch/ruleset cho org owner. +- Danh sách workflow-owner mặc định được cấu hình qua biến repository `WORKFLOW_OWNER_LOGINS` (xem CODEOWNERS cho maintainer hiện tại). - Hủy bỏ approval cũ khi có commit mới được đẩy lên. - Hạn chế force-push trên các branch được bảo vệ. +- Tất cả PR của contributor nhắm trực tiếp vào `master`. --- @@ -213,7 +215,7 @@ Chúng tôi **không** yêu cầu contributor định lượng quyền sở hữ - Mục tiêu triage maintainer đầu tiên: trong vòng 48 giờ. - Nếu PR bị chặn, maintainer để lại một checklist hành động được. - Tự động hóa `stale` được dùng để giữ hàng đợi lành mạnh; maintainer có thể áp dụng `no-stale` khi cần. -- Tự động hóa `pr-hygiene` kiểm tra các PR mở mỗi 12 giờ và đăng nhắc nhở khi PR không có commit mới trong 48+ giờ và hoặc là đang tụt hậu so với `master` hoặc thiếu/thất bại `CI Required Gate` trên head commit. +- Tự động hóa `pr-hygiene` kiểm tra các PR mở mỗi 12 giờ và đăng nhắc nhở khi PR không có commit mới trong 48+ giờ và rơi vào một trong hai trường hợp: đang tụt hậu so với `master` hoặc thiếu/thất bại `CI Required Gate` trên head commit. ### 8.1 Kiểm soát ngân sách hàng đợi