+1 Best regards, -sz
On 11/19/17, 12:51 PM, "Eric Xie" <[email protected]> wrote: Hi all, I'm starting this thread to vote on turning off protected master. The reasons are: 1. Since we turned on protected master pending PRs has grown from 40 to 80. It is severely slowing down development. 2. Committers, not CI, are ultimately responsible for the code they merge. You should only override the CI when you are very confident that CI is the problem, not your code. If it turns out you are wrong, you should fix it ASAP. This is the bare minimum requirement for all committers: BE RESPONSIBLE. I'm aware of the argument for using protected master: It make sure that master is stable. Well, master will be most stable if we stop adding any commits to it. But that's not what we want is it? Protected master hardly adds any stability. The faulty tests that breaks master at random got merged into master because they happened to succeed once. Thanks, Junyuan Xie
