On Tuesday, June 17, 2025 1:15:55 PM Mountain Standard Time Otto Kekäläinen wrote: > I just merged a first-time contributor MR in > https://salsa.debian.org/salsa-ci-team/pipeline/-/merge_requests/614, > and I think it was good quality. I don't think that people who use MRs > are noobs and their quality is in general bad, and I don't think this > barrier of e-mail-only really helps drive up contribution quality.
I completely agree with this sentiment. I know that not all the people who prefer to use an email patch workflow believe what is being discussed above. Some prefer it because it fits their taste or is what they are used to. But I have read enough comments from a minority of people who do believe that using a forge produces poorer quality code that I think it is important to emphasize that in my experience it just isn’t true. Regarding working with patch submissions, I greatly prefer an MR. I almost never merge a request without requesting changes, or at least clarification. Having the ability to highlight specific lines of code and start a thread discussing them, and having several threads going at once for different parts of the code, each of which can independently be marked as completed, is a workflow that I would find difficult to replicate in email. -- Soren Stoutner so...@debian.org
signature.asc
Description: This is a digitally signed message part.
