<div dir="ltr"><div dir="ltr">HI Julien,</div><div><br></div>Thanks for the feedback !<div><br><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">Le jeu. 12 déc. 2024 à 13:52, Julien Cabieces <<a href="mailto:julien.cabieces@oslandia.com">julien.cabieces@oslandia.com</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"><br>
Hi Denis,<br>
<br>
Thanks for this work, it works like a charm!<br>
<br>
Could you add some instructions on how to use it and set it up correctly<br>
in the developer guide ?</blockquote><div> </div><div>This is already published in <a href="https://docs.qgis.org/3.40/en/docs/developers_guide/git.html#procedure">https://docs.qgis.org/3.40/en/docs/developers_guide/git.html#procedure</a></div><div>Let me know if this needs to be improved.</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"> <br>
<br>
I think it's a great tool on developer side but I have my concerns about<br>
the CI part. It adds a lot of automatic commit which doesn't help when<br>
you want to read the git history. If you want to blame a specific line<br>
of code, you can land in this type of commit instead of the relevant<br>
modification you were heading to. You can trace it back to the pull<br>
request but it's one more step, and it makes us even more attached to the<br>
GitHub platform.<br></blockquote><div><br></div><div>Thanks to Nyall's hint, I have added the refactoring commits to <a href="https://github.com/qgis/QGIS/blob/master/.git-blame-ignore-revs">https://github.com/qgis/QGIS/blob/master/.git-blame-ignore-revs</a></div><div>When I blame files online in Github it seems to be working:</div><div>For instance, <a href="https://github.com/qgis/QGIS/blame/master/src/gui/actions/qgsactionmenu.h">https://github.com/qgis/QGIS/blame/master/src/gui/actions/qgsactionmenu.h</a></div><div>Last commit is the pre-commit run but you don't see it in the blame.</div><div><br></div><div>But I have not done this job in the release-3_40 branch (in which I manually backported).</div><div><br></div><div>More generally I am not super happy about clang-format being more painful to play with our SIP macros but it's much easier than having to build a dedicated tool. And having everything in a pre-commit config is much more convenient and clearer/cleaner.</div><div><br></div><div>Best wishes,</div><div>Denis</div><div><br></div><div><br></div><div><br></div></div></div></div>