<div dir="auto"><div>Hi, </div><div dir="auto"><br></div><div dir="auto">CC: only kdepim, kde-frameworks-devel</div><div dir="auto"><br></div><div dir="auto">I have some general comments about the API documentation. Looking at</div><div dir="auto"><a href="https://api.kde.org/stable/kdepimlibs-apidocs/kholidays/html/classes.html">https://api.kde.org/stable/kdepimlibs-apidocs/kholidays/html/classes.html</a><br></div><div dir="auto">the list mostly (only?) contains classes that are internal. Looking at </div><div dir="auto"><a href="https://github.com/KDE/kholidays/blob/master/src/holiday.h">https://github.com/KDE/kholidays/blob/master/src/holiday.h</a><br></div><div dir="auto">we can see documentation exists also for important classes, but the docs are not listed because the class documentation is missing.</div><div dir="auto"><br></div><div dir="auto">This is not a showstopper, but without accessible API, the visibility of this framework is much reduced. It would be nice, if the API documentation is complete, so here is still some room for improvement ;)</div><div dir="auto"><br></div><div dir="auto">Best regards</div><div dir="auto">Dominik</div><div dir="auto"><br></div><div dir="auto"><br><div class="gmail_extra" dir="auto"><br><div class="gmail_quote">Am 20.01.2018 11:39 schrieb "Volker Krause" <<a href="mailto:vkrause@kde.org" target="_blank">vkrause@kde.org</a>>:<br type="attribution"><blockquote class="m_-2724107121981654634quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="m_-2724107121981654634elided-text">On Sunday, 14 January 2018 12:55:30 CET David Faure wrote:<br>
> On dimanche 14 janvier 2018 10:20:38 CET Volker Krause wrote:<br>
> > On Tuesday, 6 September 2016 12:03:15 CET Volker Krause wrote:<br>
> > > On Friday 01 January 2016 18:24:17 David Faure wrote:<br>
> > > > On Thursday 24 December 2015 12:28:13 John Layt wrote:<br>
> > > > > Hi,<br>
> > > > ><br>
> > > > > It's xmas holidays, so it must be time to poke a stick at KHolidays<br>
> > > > > again<br>
> > > > > for inclusion as a Framework. As far as I am aware there are no<br>
> > > > > outstanding<br>
> > > > > porting issues with KHolidays and it is ready for review to be<br>
> > > > > included<br>
> > > > > as<br>
> > > > > a Tier 1 Framework in the next possible release. What's the next<br>
> > > > > step?<br>
> > > ><br>
> > > > Please make sure it passes all of the items in this checklist<br>
> > > > <a href="https://community.kde.org/Frameworks/CreationGuidelines" rel="noreferrer" target="_blank">https://community.kde.org/Fram<wbr>eworks/CreationGuidelines</a><br>
> > ><br>
> > > AFAICS this is followed, apart from using the KF5 version number and<br>
> > > actually being marked as a framework, which I guess is pending framework<br>
> > > approval.<br>
> ><br>
> > This got lost somehow, any objection to executing the move to frameworks<br>
> > for 5.43, say end of this week?<br>
><br>
> Go ahead.<br>
<br>
</div>The necessary metainfo and CMake changes are pushed, the sysadmin ticket for<br>
the repo metadata change is T7791.<br>
<br>
Summary: KHolidays will not be part of the 18.x KDE Application releases<br>
anymore, but instead become part of the KDE Frameworks releases with version<br>
5.43. There are no ABI or name changes, just the .so version increases from<br>
5.7 to 5.43 to match the rest of KF5, so the transition should hopefully be<br>
hardly noticeable.<br>
Side benefit: plasma-workspace no longer depends on a library from KDE<br>
Application releases.<br>
<br>
Regards,<br>
Volker<br>
</blockquote></div><br></div></div></div>