It's true that I've never seen a fake PR so far in the ASF repos but at the same time I am not following every single PR. I can understand the approval for first time contributors but keeping the approval for everyone seems a bit too much.
Best, Stamatis On Wed, Mar 29, 2023 at 5:04 AM 王鹏 <peng713...@gmail.com> wrote: > > +1 for this. I think it's very friendly to create PR for contributors. > > Dan Zou <zoud...@163.com> 于2023年3月29日周三 10:13写道： > > > +1 for this, it is something out of my expectation when I created my first > > PR in calcite and informed that approval is required. > > > > Best, > > Dan Zou > > > > > > > > > > > > > 2023年3月28日 21:37，Benchao Li <libenc...@apache.org> 写道： > > > > > > I'm +1 for changing it back. > > > > > > I was also thinking about starting a discussion about this before, but I > > > didn't know how to achieve it. Thanks Julian for sharing this with us. > > > > > > Julian Hyde <jhyde.apa...@gmail.com> 于2023年3月28日周二 20:57写道： > > > > > >> (Forwarding from the Druid list for discussion.) > > >> > > >> Julian > > >> > > >> Begin forwarded message: > > >> > > >>> From: Gian Merlino <g...@apache.org> > > >>> Date: March 28, 2023 at 1:24:42 AM CDT > > >>> To: d...@druid.apache.org > > >>> Subject: CI requiring approval for external contributors > > >>> Reply-To: d...@druid.apache.org > > >>> > > >>> Recently, ASF GitHub repos had their defaults for GitHub Actions > > >> changed to > > >>> "always require approval for external contributors". In Slack, Karan > > >>> pointed out that Airflow has recently submitted a ticket to have that > > >>> changed back: https://issues.apache.org/jira/browse/INFRA-24200. IMO, > > we > > >>> should do the same. I don't think we have a problem with fake PRs, but > > we > > >>> can always improve our responsiveness to contributors from outside the > > >>> project! Every little bit helps, including running CI automatically. > > >>> > > >>> If others have opinions on this, let me know. I'd like to raise our own > > >>> ticket to change our default. > > >>> > > >>> Gian > > >> > > > > > > > > > -- > > > > > > Best, > > > Benchao Li > > > >
