2010YOUY01 commented on code in PR #24510: URL: https://github.com/apache/datafusion/pull/24510#discussion_r4179870418
########## docs/source/contributor-guide/index.md: ########## @@ -129,6 +134,87 @@ Please ensure your PR follows the [testing guide](testing.md). In particular: [Choosing What Kind of Test to Write](testing.md#choosing-what-kind-of-test-to-write). - Run any relevant commands from the [testing quick start](testing.md#testing-quick-start). +## AI-Assisted contributions + +DataFusion has the following policy for AI-assisted PRs: + +- We welcome AI-assisted PRs from anyone. We do not welcome unreviewed "AI dumps" (defined below). +- The PR author should have personally read the entire PR they submit, and **understand the core ideas** behind the implementation **end-to-end**. Authors should be ready to justify and help reviewers understand the design and code during review. Review Comment: > i do feel the "Authors should be ready to justify and help reviewers understand the design and code during review" should hopefully cover this; 100%, this is a good practical test for 'enough undersantinding for the PR' I think it also worth a separate documentation section for 'how to write good PR/Issue description', I'll give it a try later. -- 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]
