On Mon, 23 Sept 2024 at 16:20, Florian Weimer wrote: > > * Jonathan Wakely: > > > The discussion is about how we do patch submission and patch review. > > The GitHub pull request workflow is widely seen as simpler than our > > current email-based workflow (not everybody agrees, of course). The > > idea is to *lower* the barrier of entry for contributors, not raise > > it. > > It should also help those who feel unsure about their use of Git because > the reviewers see exactly what ends up getting merged (at least with the > default workflow). With the email-based workflow, that isn't the case, > and I know it makes some people feel nervous.
Yes, our current workflow is "the patch in your email looks good, please push what you have in your tree", and what gets pushed could be different from what was reviewed (probably unintentionally). I don't think issues caused by this happen often, but a pull-based workflow can avoid the problem.
