GitHub user zhztheplayer edited a discussion: Infra: Remove the approval restriction of merging PRs
Hi folks, You may notice that in our GitHub repo, at least one approval from committers is needed before a PR is mergeable. This setting is configured at [here](https://github.com/apache/incubator-gluten/blob/b09c017dffb4246dc4b29d8cbe6481fa59a50d18/.asf.yaml#L41). I would like to explore on the possibility of removing this rule. There are a lot of other Apache projects not having this rule but still running perfectly, including the well-known ones like Hadoop, Spark, Arrow, etc. Actually, I found the number of repositories from ASF projects having this restriction is way less than the number of others that don't have this restriction. See my search result: 1. Number of repositories with .asf.yaml defined: [~1100](https://github.com/search?q=org%3Aapache+path%3A%22.asf.yaml%22+%22github%22&type=code&p=5) [1] 2. Number of repositories with .asf.yaml defined with the approval restriction: [228](https://github.com/search?q=org%3Aapache+path%3A%22.asf.yaml%22+%22required_approving_review_count%22&type=code) Hence, I assume in most case a Apache project can run well without this restriction. Removing the restriction doesn't mean PRs can be arbitrarily merged without essential reviews: It's always the committer's duty to consider carefully on whether review is needed for their PR before merging, One may says people make mistakes, I think that's pretty normal however when mistake happens, we can always revert the unwanted changes to fix. As only committers can merge PRs, I think the risk can be controllable given we always carefully nominate committers. We can observe on the overall cost after the rule change is applied. As the change can be experimental, I would recommend to remove the rule and observe on the impact for next several weeks, to see if the new rule is suitable for the community. Do you have any thoughts? cc @apache/gluten-committers [1] Note, ASF doesn't have this many of TLPs. The number includes all the sub-repositories under the TLPs (and incubating ones as well). GitHub link: https://github.com/apache/incubator-gluten/discussions/8556 ---- 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]
