This is an automated email from the ASF dual-hosted git repository.
wu-sheng pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/skywalking-horizon-ui.git
The following commit(s) were added to refs/heads/main by this push:
new c20c8f7 infra: enable branch protection on main
c20c8f7 is described below
commit c20c8f79c4458d4b182785d4fcf02e9130475624
Author: Wu Sheng <[email protected]>
AuthorDate: Thu May 21 23:26:15 2026 +0800
infra: enable branch protection on main
Require the rolled-up "Required" CI status check (strict — branch must be
up to date before merge) and one approving PR review before merging to main.
---
.asf.yaml | 19 ++++++++++++-------
1 file changed, 12 insertions(+), 7 deletions(-)
diff --git a/.asf.yaml b/.asf.yaml
index 8ddaeba..790c91a 100644
--- a/.asf.yaml
+++ b/.asf.yaml
@@ -34,10 +34,15 @@ github:
merge: false
rebase: false
dependabot_updates: false
- # Branch protection and PR approval are intentionally OFF during pre-1.0
- # single-author development to keep the iteration loop tight. The explicit
- # `~` (YAML null) tells ASF infra to clear any previously-applied
- # `protected_branches` config — omitting the key would leave prior
- # protections in place. Re-add status-check + review gating before v1.0
- # or when a second committer joins.
- protected_branches: ~
+ protected_branches:
+ main:
+ required_status_checks:
+ # Require branches to be up to date with main before merging.
+ strict: true
+ # The CI "required" job rolls up every check (license-header,
+ # dependency-license, type-check, build-ui, build-bff, test) into a
+ # single "Required" status, so protection gates on one context.
+ contexts:
+ - Required
+ required_pull_request_reviews:
+ required_approving_review_count: 1
