On Wed, 18 Oct 2023 at 06:40, David Rowley <dgrowle...@gmail.com> wrote: > How many of the committers who have broken koel are repeat offenders?
I just checked the commits and there don't seem to be real repeat offenders. The maximum number of times someone broke koel since its inception is two. That was the case for only two people. The other 8 people only caused one breakage. > What is their opinion on this? > Did they just forget once or do they hate the process and want to go back? The commiters that broke koel since its inception are: - Alexander Korotkov - Amit Kapila - Amit Langote - Andres Freund - Etsuro Fujita - Jeff Davis - Michael Paquier - Peter Eisentraut - Tatsuo Ishii - Tomas Vondra I included all of them in the To field of this message, in the hope that they share their viewpoint. Because otherwise it stays guessing what they think. But based on the contents of the fixup commits a commonality seems to be that the fixup only fixes a few lines, quite often touching only comments. So it seems like the main reason for breaking koel is forgetting to re-run pgindent after some final cleanup/wording changes/typo fixes. And that seems like an expected flaw of being human instead of a robot, which can only be worked around with better automation. > I agree that it's not nice to add yet another way of breaking the > buildfarm and even more so when the committer did make check-world > before committing. We have --enable-tap-tests, we could have > --enable-indent-checks and have pgindent check the code is correctly > indented during make check-world. Then just not have > --enable-indent-checks in CI. I think --enable-indent-checks sounds like a good improvement to the status quo. But I'm not confident that it will help remove the cases where only a comment needs to be re-indented. Do commiters really always run check-world again when only changing a typo in a comment? I know I probably wouldn't (or at least not always).
