I will update labels only based on committer's approvals (for everything),
I think that's cleaner.

We can always revisit this.

On Wed, Feb 27, 2019 at 4:31 PM Chesnay Schepler <ches...@apache.org> wrote:

> Fore code-quality/description I agree, but consensus and the final
> approval should require a committer IMO.
>
> On 27.02.2019 15:08, Robert Metzger wrote:
>
> I did not put any restrictions on who can communicate with the bot!
> But since there is currently no way of overriding somebody's approval for
> something, this can easily lead to such a situation.
>
> My thinking was that a committer still needs to manually check who
> approved a pull request, and I wanted to be open for non-committers to
> participate in the review process.
> WIth the labels in place, this can easily send the wrong message.
>
> What should we do?
> A) we restrict sending commands to the bot to committers?
> B) only approvals from committers matter for applying labels?
> C) we allow committers to override approvals
>
> I'm leaning towards B, as it encourages non-committers to participate.
>
>
> On Wed, Feb 27, 2019 at 2:01 PM Chesnay Schepler <ches...@apache.org>
> wrote:
>
>> Just noticed that _anyone_ can approve a PR now, see
>> https://github.com/apache/flink/pull/7801.
>>
>> Not sure about the solution, but as it stands it is rather trivial to
>> nuke the review process of the entire project.
>>
>> On 13.02.2019 10:29, Robert Metzger wrote:
>> > Hey all,
>> >
>> > the flinkbot has been active for a week now, and I hope the initial
>> hiccups
>> > have been resolved :)
>> >
>> > I wanted to start this as a permanent thread to discuss problems and
>> > improvements with the bot.
>> >
>> > *So please post here if you have questions, problems or ideas how to
>> > improve it!*
>> >
>>
>>
>

Reply via email to