nevzheng opened a new issue, #12795:
URL: https://github.com/apache/gravitino/issues/12795

   ### What would you like to be improved?
   
   Gravitino already has Copilot review instructions, and reviewers can request 
Copilot manually, but pull requests do not get an automatic first-pass review.
   
   Reviewer bandwidth is limited. Apache Iceberg, Arrow, Polaris, and Traffic 
Server already turned this on through ASF-managed `.asf.yaml`. Iceberg's 
discussion is here: 
https://lists.apache.org/thread/n2s6z7rt745cgftsb8j9yob0kc70cgny
   
   ### How should we improve?
   
   Enable ASF-managed GitHub Copilot code review on `apache/gravitino`:
   
   - `github.copilot_code_review.enabled: true`
   - `review_drafts: false` (same as Arrow's later tuning)
   - `review_on_push: true`
   
   Reuse the existing `.github/copilot-instructions.md`. Document in 
`CONTRIBUTING.md` that Copilot comments are first-pass hints: implement them or 
reply why not. Human reviewers remain responsible for the merge decision.
   
   This creates a GitHub ruleset named `Copilot Code Review` on the default 
branch. It does not grant Copilot licenses.
   
   - Automatic review AI credits are billed to the PR author.
   - Manual review credits are billed to the requester.
   - Authors without Copilot code review access will not get an automatic 
review; anyone with access can still request one.
   - Agentic review uses GitHub Actions minutes on the repository (public repos 
have a free Actions allowance).
   
   ASF Infra asks projects to discuss `.asf.yaml` workflow impact before 
enabling it: 
https://github.com/apache/infrastructure-asfyaml#before-you-start-using-asfyaml
   
   Nevin
   Sent from my 🤖 (Cursor)


-- 
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]

Reply via email to