github-actions[bot] commented on code in PR #66165:
URL: https://github.com/apache/doris/pull/66165#discussion_r3665097766


##########
.asf.yaml:
##########
@@ -192,9 +192,9 @@ github:
   collaborators:
     - Yukang-Lian
     - Hastyshell
-    - yujun777
-    - ixzc
     - feiniaofeiafei
+    - ixzc
+    - yujun777

Review Comment:
   **This order-only edit cannot retrigger the missing collaborator 
invitations.** The ASF handler converts both the configured and cached 
collaborator lists to sets and only calls `add_to_collaborators` when those 
sets differ ([current 
handler](https://github.com/apache/infrastructure-asfyaml/blob/dc85eb772224e05378df468177555a6253d83e8d/asfyaml/feature/github/collaborators.py#L32-L65)).
 PR #65980 already made a real set change and wrote this same eight-user cache; 
live `affiliation=direct` state now contains the other five accounts with 
`triage`, but still omits `feiniaofeiafei`, `ixzc`, and `yujun777`. Because 
this patch preserves the set, it makes no collaborator API call. ASF's 
[official 
guidance](https://infra.apache.org/blog/newsletter_05_25.html#did-you-know) 
requires removal/push and then re-addition/push as two separate changes to 
renew expired invitations. Please use that sequence or ask ASF Infrastructure 
to reissue the invitations.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to