<div dir="ltr"><div>PR 1527 introduces Black to wxGUI code. Open PRs will likely need to be updated before merging.<br></div><div><br></div><div><a href="https://github.com/OSGeo/grass/pull/1527">https://github.com/OSGeo/grass/pull/1527</a></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Feb 26, 2021 at 11:24 PM 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 dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Feb 26, 2021 at 7:01 AM Luca Delucchi <<a href="mailto:lucadeluge@gmail.com" target="_blank">lucadeluge@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">On Fri, 26 Feb 2021 at 06:00, Vaclav Petras <<a href="mailto:wenzeslaus@gmail.com" target="_blank">wenzeslaus@gmail.com</a>> wrote:<br><br>
> All this applies to core repo, not the grass-addons repo. Ideas and volunteers are welcome.<br>
><br>
<br>
Do you have any suggestions on how to proceed to grass-addons?<br>
Did you just run blake inside a folder or did you do something more?<br></blockquote><div><br></div><div>Here is what I did:<br></div><div><br></div><div>* Applied Black.<br></div><div>* Added pyproject.toml with configuration for Black.</div><div>* Modified Flake8 rules.</div><div>* Added a Black GitHub workflow.</div><div><br></div><div>The application of Black can use some touches here and there. You need to use the same Black version. You may want to modify some lines before and after (I did just things I caught for one reason or another - you cannot review all changes). You may run Flake8 to check for long lines after you applied Black and see if there is something you can resolve. I did separate PRs for different directories, but that was because the codebases are different in each and also a little bit to minimize conflicts with other PRs (these should not be an issue with grass-addons repo). Commit per directory, but then squashing them when merging the PR might be a good approach for grass-addons.<br></div></div></div>
</blockquote></div>