alamb commented on code in PR #17183:
URL: https://github.com/apache/datafusion/pull/17183#discussion_r2276303527
##########
.asf.yaml:
##########
@@ -50,8 +50,38 @@ github:
main:
required_pull_request_reviews:
required_approving_review_count: 1
+ required_status_checks:
+ strict: true # Require branches to be up to date before merging
Review Comment:
Can we not require up to date branches before we have enabled the merge
queue? Otherwise that will slow down merging of PRs as we'll have a bottleneck
/ race for each PR that wants to merge
Once there is an automated queue it makes sense because the queue handles
the race
--
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: github-unsubscr...@datafusion.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org
For additional commands, e-mail: github-h...@datafusion.apache.org
