mbutrovich commented on code in PR #7135: URL: https://github.com/apache/datafusion-comet/pull/7135#discussion_r4327436580
########## docs/source/contributor-guide/development.md: ########## @@ -633,6 +633,12 @@ It is possible to debug both native and JVM code concurrently as described in th ## Submitting a Pull Request +With coding agents, the number of open PRs can far exceed our review capacity. To help reviewers +focus on fewer PRs at a time, users without write access are limited to **3 open, non-draft PRs**. +If you reach the limit, wait for existing PRs to be merged, close them, or convert them to drafts +before opening another. Prefer a single PR with one commit per part over a series of small PRs +that are opened at the same time. Review Comment: This wording seems to run counter to wanting small, reviewable PRs and might have the unintended effect of resulting in more monolithic PRs. I think we should still encourage multiple PRs in sequence, but the one that's ready for review/the dependency for later PRs should be the only one non-draft. This is especially important because commits get squashed anyway, so saying we should open one PR with commits for each part doesn't make sense for the changelog, and reverting gets trickier. -- 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]
