This is an automated email from the ASF dual-hosted git repository.

pan3793 pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/celeborn.git


The following commit(s) were added to refs/heads/main by this push:
     new 52fa5ffb8 [CELEBORN-2438][INFRA] Remove empty protected_branches in 
.asf.yaml
52fa5ffb8 is described below

commit 52fa5ffb85221c9b8d2907352620860f83b1dd5e
Author: Cheng Pan <[email protected]>
AuthorDate: Thu Aug 20 22:44:24 2026 +0800

    [CELEBORN-2438][INFRA] Remove empty protected_branches in .asf.yaml
    
    ### What changes were proposed in this pull request?
    
    Remove the empty protected_branches key from `.asf.yaml`.
    
    ### Why are the changes needed?
    
    ASF infra validation rejects .asf.yaml because github.protected_branches
    expects a mapping, and the empty value fails schema parsing. Branch
    protection is already handled by github.rulesets, so the key is redundant.
    
    ### Does this PR resolve a correctness bug?
    
    - [ ] Yes
    
    ### Does this PR introduce _any_ user-facing change?
    
    - [ ] Yes
    
    ### How was this patch tested?
    
    Parsed .asf.yaml with a YAML parser to confirm it loads.
    
    Assisted-by: DeepSeek V4 Pro
    
    Closes #3819 from pan3793/CELEBORN-2438.
    
    Authored-by: Cheng Pan <[email protected]>
    Signed-off-by: Cheng Pan <[email protected]>
---
 .asf.yaml | 1 -
 1 file changed, 1 deletion(-)

diff --git a/.asf.yaml b/.asf.yaml
index 0e0ec729b..20c47a333 100644
--- a/.asf.yaml
+++ b/.asf.yaml
@@ -28,7 +28,6 @@ github:
     squash: true
     merge: false
     rebase: false
-  protected_branches:
   features:
     issues: true
     discussions: true

Reply via email to