<div dir="ltr"><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Apr 6, 2021 at 11:08 AM Harald Sitter <<a href="mailto:sitter@kde.org">sitter@kde.org</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">- I'm like a broken record at this point: some of the cmake files and<br>
the json file lack licensing details. please run `reuse lint` to check<br>
the list. it's incredibly close to being fully licensed<br>
- you include KDEClangFormat but don't actually seem to enable it<br>
- along a similar note: the code style is inconsistent at times, you<br>
might want to also enable the githooks when built with a new enough<br>
ECM<br>
- CheckIncludeFiles is also included but not used<br>
- you sometimes use the Qt5::Foo targets rather than the Qt::Foo<br>
targets, I believe the latter is preferred for easier porting to qt6<br>
- the library code calls qputenv and ignores its return value, might<br>
make sense to qcwarning when the env variable failed to set<br>
<br>
other than that it looks lovely 👍<br>
<br>
HS<br></blockquote><div><br></div><div style="font-size:small" class="gmail_default">I think all the issues brought up here have been addressed.</div><div style="font-size:small" class="gmail_default"><br></div><div style="font-size:small" class="gmail_default">Thanks everyone!</div><div style="font-size:small" class="gmail_default">Aleix</div></div></div>