<div dir="ltr"><div dir="ltr">On Tue, 13 Sept 2022 at 09:33, Vaclav Petras <<a href="mailto:wenzeslaus@gmail.com">wenzeslaus@gmail.com</a>> wrote:<br></div><div class="gmail_quote"><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"><div></div><div><br></div><div>Formatting needs to be enforced in the CI. This can be done with something like re-indent followed by git diff.<br></div><div><br></div></div></blockquote><div><br></div><div>Maybe something like this could be used? <a href="https://github.com/marketplace/actions/clang-format-check">https://github.com/marketplace/actions/clang-format-check</a>  Note that it doesn't actually format the code, but would complain like the current black formatter for python 🤔</div><div><br></div><div>or perhaps <a href="https://github.com/material-foundation/clang-format-ci">https://github.com/material-foundation/clang-format-ci</a> seems to do the same, and leaves inline comments</div><div><br></div><div>--Wolf</div><div><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"><div></div><div>Best,<br></div><div>Vaclav<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, 27 Aug 2022 at 14:06, Vaclav Petras <<a href="mailto:wenzeslaus@gmail.com" target="_blank">wenzeslaus@gmail.com</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"><div>Dear all,</div><div><br></div><div>Most of the C and C++ code was indented using our ./utils/grass_indent_ALL.sh script at the FOSS4G 2022 sprint. This will help to indent code for PRs and with the transition to clang-format.<br></div><div><br></div><div>Directories with files which indent reports issues for were not updated, because more changes are needed in the code.<br></div><div><br></div><div>This may create conflicts in some existing PR which modify C code. If you have a conflict related to this (old PR may have other conflicts too), you can follow this procedure:<br><br>Indent your code:<br><br>./utils/grass_indent.sh db/drivers/mysql/create_table.c<br><br>Commit the change:<br><br>git commit -m "Indent PR code" db/drivers/mysql/create_table.c<br><br>Update upstream/main branch:<br><br>git fetch upstream<br><br>Merge into your branch upstream/main:<br><br>git merge --strategy=recursive --strategy-option=ours upstream/main</div><div><br></div><div>Let me know if you have any questions,</div><div>Vaclav<br></div></div>
</blockquote></div>
_______________________________________________<br>
grass-dev mailing list<br>
<a href="mailto:grass-dev@lists.osgeo.org" target="_blank">grass-dev@lists.osgeo.org</a><br>
<a href="https://lists.osgeo.org/mailman/listinfo/grass-dev" rel="noreferrer" target="_blank">https://lists.osgeo.org/mailman/listinfo/grass-dev</a><br>
</blockquote></div></div>