LGTM now +2 On Wed, Jun 21, 2023 at 10:04=E2=80=AFAM Marco Martin w= rote: > > I fixed CI, passes now > > On Tue, Jun 20, 2023 at 8:44=E2=80=AFPM Ben Cooksley = wrote: > > > > Hi all, > > > > Sysadmin has just received a request to move this repository to Framewo= rks, however after seeing some of the comments raised here regarding the re= pository I took a look myself to see if they had been corrected. > > > > At this time CI is still broken in KSvg, for both platforms - something= that was mentioned in an earlier email here. > > This does not inspire confidence that the code is up to scratch. > > > > I've therefore declined to move it to Frameworks at this time. > > > > Would be appreciated, given this is looking to be promoted to Framework= s, if people could please have a further look at this repository and commen= t as appropriate. > > > > Thanks, > > Ben > > > > On Wed, Jun 14, 2023 at 9:12=E2=80=AFPM Marco Martin wrote: > >> > >> Hi all, > >> Some time has passes, and changes have been done in the repo to > >> address some of the points. > >> Now there are review requests in plasma-framework which depends on > >> this repo (and accompanying plasma-workspace, plasma-desktop etc) > >> It would probably be better to have this in frameworks to have the > >> rest depending from it? > >> > >> On Thu, Apr 20, 2023 at 10:25=E2=80=AFAM Marco Martin wrote: > >> > > >> > Hi all, > >> > A part of plasma-framewrok, which is the one to do SVG-based themes, > >> > has now been splitted in a standalone library which is intended to b= e > >> > a new framework in KF6 (all usages of the plasma-framework version > >> > will be ported once this officially lands, and then those classes wi= ll > >> > be removed) > >> > The repo for now lives in > >> > https://invent.kde.org/libraries/plasmasvg > >> > > >> > In the end it will be renamed in ksvg > >> > > >> > Comments? reviews? > >> > > >> > > >> > -- > >> > Marco Martin > >> > >> -- > >> Marco Martin