2010YOUY01 commented on issue #25317: URL: https://github.com/apache/datafusion/issues/25317#issuecomment-5676691866
Another idea that came to mind is a checklist-based review. 🤔 I remember someone previously proposed a contributor PR checklist, but it got pushed back because such a checklist tends to grow very long, and eventually no one reads it. However, this seems worth maintaining if we can trigger LLM reviews cheaply. The idea: we maintain a checklist for PRs (e.g., the testing policy), and the AI reviewer only points out violations of the checklist. Since each item maps to a known rule, these reviews are much easier to understand than open-ended ones. And because the checklist is read by the LLM rather than by every contributor, it can afford to be long. -- 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]
