<div dir="auto"><div><br><br><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">Le jeu. 12 déc. 2024, 22:22, Stefanos Natsis <<a href="mailto:uclaros@gmail.com">uclaros@gmail.com</a>> a écrit :<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>I also don't like the commit bot.</div><div>I've already had to force push, which I don't like to as it usually disturbs reviewing, because it committed a change even though pre-commit was ran locally!</div><div><a href="https://github.com/qgis/QGIS/pull/59790/commits/4fc5d7011b8dbb234850f7147b69825d275f6f66" target="_blank" rel="noreferrer">https://github.com/qgis/QGIS/pull/59790/commits/4fc5d7011b8dbb234850f7147b69825d275f6f66</a></div></div></blockquote></div></div><div dir="auto"><br></div><div dir="auto">Clang-format was added to gui after the last commit of your head branch but before you created the pull request. So this is due to the transition and won't happen again.</div><div dir="auto"><br></div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote gmail_quote_container"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><br></div><div>Stefanos<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, 12 Dec 2024 at 21:41, Denis Rouzaud via QGIS-Developer <<a href="mailto:qgis-developer@lists.osgeo.org" target="_blank" rel="noreferrer">qgis-developer@lists.osgeo.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="ltr">I would argue in favor of the auto update:<div><br></div><div>* it's easier for newcomers: having a pull request rejected because of indentation is not welcoming</div><div>* it's easier for online editing</div><div>* the argument that you need to rebase is not valid to me:</div><div>    * you can always force push and the code beautifier will be run on top again</div><div>    * the solution you mention is to have an environment properly set up: if you do so, you won't have pre-commit commits in the log</div><div><br></div><div><br></div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Le jeu. 12 déc. 2024 à 16:53, Jean Felder via QGIS-Developer <<a href="mailto:qgis-developer@lists.osgeo.org" target="_blank" rel="noreferrer">qgis-developer@lists.osgeo.org</a>> a écrit :<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Le 12/12/2024 à 16:38, Even Rouault via QGIS-Developer a écrit :<br>
><br>
><br>
> Le 12/12/2024 à 16:33, Julien Cabieces via QGIS-Developer a écrit :<br>
>> Not a great fan of this approach, It looks like complicated to me, I would prefer the old behavior.<br>
>><br>
>> Those automatic PR force also contributor to rebase before pushing new<br>
>> modification to their branch. And if they have modified area of code<br>
>> which have been reformatted, they get conflicts.<br>
>><br>
>> automatic pre-commit commits brings more difficulties than what it<br>
>> solves IMHO.<br>
><br>
> I also tend to agree on that. On other projects I'm involved too <br>
> (GDAL, PROJ, shapelib, libtiff) where we have put in place pre-commit, <br>
> we just have a CI check that checks the formatting is OK. This <br>
> requires a bit of education to first time contributors, but pointing <br>
> to the documentation with clear instructions on how to setup <br>
> pre-commit should be good enough. <br>
> <a href="https://github.com/qgis/QGIS/edit/master/.github/PULL_REQUEST_TEMPLATE.md" rel="noreferrer noreferrer" target="_blank">https://github.com/qgis/QGIS/edit/master/.github/PULL_REQUEST_TEMPLATE.md</a> <br>
> could also have some words about it<br>
><br>
> Like in <br>
> <a href="https://github.com/OSGeo/gdal/edit/master/.github/PULL_REQUEST_TEMPLATE.md" rel="noreferrer noreferrer" target="_blank">https://github.com/OSGeo/gdal/edit/master/.github/PULL_REQUEST_TEMPLATE.md</a> <br>
> , " - [ ] Make sure code is correctly formatted (cf [pre-commit <br>
> configuration](<a href="https://gdal.org/development/dev_practices.html#commit-hooks)" rel="noreferrer noreferrer" target="_blank">https://gdal.org/development/dev_practices.html#commit-hooks)</a>)"<br>
><br>
On second thoughts, I agree with Julien and Even.<br>
<br>
Jean<br>
<br>
<br>
_______________________________________________<br>
QGIS-Developer mailing list<br>
<a href="mailto:QGIS-Developer@lists.osgeo.org" target="_blank" rel="noreferrer">QGIS-Developer@lists.osgeo.org</a><br>
List info: <a href="https://lists.osgeo.org/mailman/listinfo/qgis-developer" rel="noreferrer noreferrer" target="_blank">https://lists.osgeo.org/mailman/listinfo/qgis-developer</a><br>
Unsubscribe: <a href="https://lists.osgeo.org/mailman/listinfo/qgis-developer" rel="noreferrer noreferrer" target="_blank">https://lists.osgeo.org/mailman/listinfo/qgis-developer</a><br>
</blockquote></div>
_______________________________________________<br>
QGIS-Developer mailing list<br>
<a href="mailto:QGIS-Developer@lists.osgeo.org" target="_blank" rel="noreferrer">QGIS-Developer@lists.osgeo.org</a><br>
List info: <a href="https://lists.osgeo.org/mailman/listinfo/qgis-developer" rel="noreferrer noreferrer" target="_blank">https://lists.osgeo.org/mailman/listinfo/qgis-developer</a><br>
Unsubscribe: <a href="https://lists.osgeo.org/mailman/listinfo/qgis-developer" rel="noreferrer noreferrer" target="_blank">https://lists.osgeo.org/mailman/listinfo/qgis-developer</a><br>
</blockquote></div>
</blockquote></div></div></div>