adriangb commented on code in PR #18237: URL: https://github.com/apache/datafusion/pull/18237#discussion_r2453928400
########## docs/source/contributor-guide/index.md: ########## @@ -175,3 +175,27 @@ The good thing about open code and open development is that any issues in one ch Pull requests will be marked with a `stale` label after 60 days of inactivity and then closed 7 days after that. Commenting on the PR will remove the `stale` label. + +## AI-Assisted contributions + +For AI-assisted PRs, it is recommended to follow these policies: + +- The PR author should **understand the core ideas** behind the implementation **end-to-end**, and is able to defend the design and code during review. +- **Calls out unknowns and assumptions**. Missing some corners in a AI-generated solution is fine (e.g. not sure API call X is working for edge case Y), pointing it out can help the review focus) Review Comment: ```suggestion - **Calls out unknowns and assumptions**. It's okay to not fully understand some bits of AI generated code. You should comment on these cases and point them out to reviewers so that they can use their knowledge of the codebase to clear up any concerns. For example, you might comment "calling this function here seems to work but I'm not familiar with how it works internally, I wonder if there's a race condition if it is called concurrently". ``` I mainly wanted to make the example more concrete. -- 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]
