<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, 17 Sept 2022 at 04:57, Wolf Bergenheim <<a href="mailto:wolf%2Bgrass@bergenheim.net">wolf+grass@bergenheim.net</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 dir="ltr">On Tue, 13 Sept 2022 at 09:33, Vaclav Petras <<a href="mailto:wenzeslaus@gmail.com" target="_blank">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" target="_blank">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" target="_blank">https://github.com/material-foundation/clang-format-ci</a> seems to do the same, and leaves inline comments</div></div></div></blockquote><div><br></div><div>Yes, please. We need to transition to clang-format and one of these clang-format wrappers for CI will be needed. You can open an experimental PR if you want. See also #2272.<br></div><div><br></div><div><a href="https://github.com/OSGeo/grass/pull/2272">https://github.com/OSGeo/grass/pull/2272</a></div></div></div>