<div dir="ltr">In all honesty formatting is something that should be dealt with automatically. I use clang-format as part of my IDE just so I don't have to think about formatting in addition to the actual functionality of the code I'm writing. Auto formatting as you type helps a lot in keeping code consistently formatted even when its largely one person working on the project.<br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Apr 13, 2022 at 8:38 AM Helio Chissini de Castro <<a href="mailto:helio@kde.org">helio@kde.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="auto">+1 <div dir="auto"><br></div><div dir="auto">Só far, simple solution and would reduce a lot work of reviewers.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Apr 13, 2022, 04:27 Ingo Klöcker <<a href="mailto:kloecker@kde.org" target="_blank">kloecker@kde.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Samstag, 2. April 2022 11:21:11 CEST Kevin Kofler wrote:<br>
> Nate Graham wrote:<br>
> > This caused a problem recently in KWin. A new contributor was given<br>
> > commit rights very soon after he appeared, and then immediately after<br>
> > that, he inappropriately merged a not-fully-reviewed an un-accepted<br>
> > merge request<br>
> > (<a href="https://invent.kde.org/plasma/kwin/-/merge_requests/1980" rel="noreferrer noreferrer" target="_blank">https://invent.kde.org/plasma/kwin/-/merge_requests/1980</a>). It seems<br>
> > that he did not have a sense of the cultural norms around committing to<br>
> > KDE repos, and giving him commit access was probably premature.<br>
> <br>
> Well, the question this calls for is why the merge request was still not<br>
> fully reviewed almost six weeks after submission. I would guess that that is<br>
> what the misunderstanding came from: the submitter most likely assumed that<br>
> the changes were fine given that there were no outstanding comments. (The<br>
> submitter did try to address those comments that you had in those six<br>
> weeks.)<br>
> <br>
> I should also point out that the complaints in Xaver Hugl's post-merge<br>
> review were all only formatting/whitespace, choice of comment sign, and<br>
> brace issues (with no effect on the end user at all),<br>
<br>
Several PIM libraries have clang-format pre-commit hooks that prevent <br>
formatting issues in the first place (and, occasionally, annoy me because the <br>
hooks also complain about formatting issues in unstaged/uncommitted code, e.g. <br>
temporarily commented out code where the commented out code is not correctly <br>
indented).<br>
<br>
Formatting is something no reviewer should have to waste brain energy on <br>
nowadays.<br>
<br>
Regards,<br>
Ingo<br>
</blockquote></div>
</blockquote></div>