<div dir="ltr"><div></div><div>I have resolved all indentation-related conflicts in the open PRs. The procedure worked well on all PRs except one which had changes in directories which were not indented yet.</div><div><br></div><div>The remaining issues are:</div><div><br></div><div>There are directories where indent reports an error, esp. due to ambiguities in the current code formatting. These are not indented yet. These would be best resolved by separate PRs, possibly separating the ambiguity fixes from the overall formatting changes (esp. given that we want to ignore purely formatting changes in git blame).</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>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">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>