> On Nov 1, 2025, at 5:05 PM, Piotr P. Karwasz <[email protected]>
> wrote:
>
> Hi Phil,
>
>> On 1.11.2025 22:29, Phil Steitz wrote:
>> As PMC members it is our collective responsibility to review
>> commits. Having them bundled into PRs doesn't make that any easier,
>> IMO. At the end of the day, the commits are the changes. The best
>> thing we can do - not just here, but in every OSS community - is to
>> get more eyeballs on code. That means building communities.
>> Byzantine processes make that harder.
>
> It’s great to have many eyes on the code, but in practice, that doesn’t
> seem to be happening. Either every commit is flawless, or nobody is
> actually reviewing them, since responses on `commits@` are extremely rare.
>
> The main problem with relying solely on `commits@` is that there’s no
> visibility into whether a commit has already been reviewed (and silently
> approved) or if it hasn’t been looked at at all. In reality, it’s very
> likely that many commits go through without any real review.
Agreed that is a big problem. Self-merged PRs won’t help that.
Phil
>
> Piotr
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [email protected]
> For additional commands, e-mail: [email protected]
>
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
