alamb opened a new pull request, #24051: URL: https://github.com/apache/datafusion/pull/24051
## Which issue does this PR close? - https://github.com/apache/datafusion/issues/23839 - related to the disucssions tarted by @jayzhan211 in https://github.com/apache/datafusion/issues/21038 ## Rationale for this change As our project grows both in terms of number of users as well as the number of PRs submitted (due to agents and incrasing usage) I am trying to document / automate as much as possible As one of the largest bottlenecks at the moment is PR review, I would like to document the review process as much as posisble so that by the time commiters see a PR it is in as good shape as possible. I also strongly believe effective documentation should be written for **both** humans and agents so I purposely didn't make a specific skill for this (instead I made a skill that is an agent) ## What changes are included in this PR? 1. Add a new PR review page to the contributor guide 2. Try and distill project best practice 3. Leave links to help people/agents find it ## Are these changes tested? By CI ## Are there any user-facing changes? <!-- If there are user-facing changes then we may require documentation to be updated before approving the PR. --> <!-- If there are any breaking changes to public APIs, please add the `api change` label. --> -- 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]
