GitHub user zhztheplayer added a comment to the discussion: Infra: Remove the approval restriction of merging PRs
> +1 for zhouyuan’s opinion. I think we still need additional committer > approvals before code can be merged. In the past, we’ve had several instances > where code was reverted after being merged due to objections. If we remove > this restriction, I am concerned that more problems will occur. I understand, so I recommend to try with no restrictions for several weeks to see if we have more cost because of the change. If we do see many PRs were reverted because of no reviews, we could add the rule back. Does that sound work for you? It will be worth it if we find the change doesn't bring burden (e.g., many code reverts) to the community. GitHub link: https://github.com/apache/incubator-gluten/discussions/8556#discussioncomment-11865414 ---- This is an automatically sent email for [email protected]. To unsubscribe, please send an email to: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
