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. Piotr --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
