<div dir="ltr"><div dir="ltr"><div>Aleix,</div><div><br></div><div>Yes, I know frameworks have lower dependencies - I just wanted to fix QML warnings in Plasma Workspace. The question is:</div><div>Is it OK to do this in one bug change, or should I split it into smaller?<br></div><div><br></div><div>Konrad<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">pon., 9 lis 2020 o 14:23 Aleix Pol <<a href="mailto:aleixpol@kde.org">aleixpol@kde.org</a>> napisał(a):<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Konrad,<br>
It's okay to fix them in Plasma. Most of the ones we still have though<br>
are in frameworks that still depend on Qt 5.12 and it's not trivial to<br>
address this problem over there.<br>
<br>
Aleix<br>
<br>
On Mon, Nov 9, 2020 at 10:25 AM Konrad Materka <<a href="mailto:materka@gmail.com" target="_blank">materka@gmail.com</a>> wrote:<br>
><br>
> Hi,<br>
><br>
> With Qt 5.15 we have tons of warnings like:<br>
> QML Connections: Implicitly defined onFoo properties in Connections are deprecated.<br>
><br>
> Qt 5.15 is now a minimum requirement for Plasma Workspace, so nothing is blokcking to migrate to new syntax.<br>
><br>
> Is it OK to fix them in one massive MR or better to do that one by one (I mean one applet/module at a time)?<br></blockquote></div></div>