D29667: Export metalist to json file

ochurlaud accepted this revision. ochurlaud added a comment. This revision is now accepted and ready to land. Alright! Ship it REPOSITORY R264 KApiDox BRANCH cblack/metadata (branched from master) REVISION DETAIL https://phabricator.kde.org/D29667 To: cblack, ochurlaud Cc: bcooksley,

D29667: Export metalist to json file

ochurlaud added a comment. Don't use file as a variable name as it's a python keyword. Do you really want to dump those files everywhere? REPOSITORY R264 KApiDox REVISION DETAIL https://phabricator.kde.org/D29667 To: cblack, ochurlaud Cc: bcooksley, kde-frameworks-devel,

Re: Information regarding upcoming Gitlab Migration

Le 27 avril 2020 22:33:12 GMT+02:00, Ben Cooksley a écrit : >On Tue, Apr 28, 2020 at 8:31 AM Albert Astals Cid >wrote: >> >> El dilluns, 27 d’abril de 2020, a les 13:19:07 CEST, Ben Cooksley va >escriure: >> > On Mon, Apr 27, 2020 at 11:12 PM Olivier Chur

Re: Information regarding upcoming Gitlab Migration

Hi, Le lundi 27 avril 2020, 12:38:42 CEST Aleix Pol a écrit : > On Mon, Apr 27, 2020 at 3:41 AM Bhushan Shah wrote: > > > > [Please keep sysad...@kde.org list or bs...@kde.org in the CC for > > replies] > > > > Hello Community members, > > > > In view of upcoming Gitlab migration, we sysadmin

Re: Information regarding upcoming Gitlab Migration

Le lundi 27 avril 2020, 13:19:07 CEST Ben Cooksley a écrit : > On Mon, Apr 27, 2020 at 11:12 PM Olivier Churlaud > wrote: > > > > Hi, > > > > Le lundi 27 avril 2020, 12:38:42 CEST Aleix Pol a écrit : > > > On Mon, Apr 27, 2020 at 3:41 AM Bhushan Shah wr

D28811: Add drawer on mobile and clean code

ochurlaud accepted this revision. ochurlaud added a comment. In D28811#649209 , @ognarb wrote: > I now generate the CSS from aether-sass. The code can be found https://invent.kde.org/websites/aether-sass/-/blob/master/css/aether-sidebar.scss.

D28811: Add drawer on mobile and clean code

ochurlaud accepted this revision. ochurlaud added a comment. OK it's clearer: Please add somewhere that it's a generated css so that no-one tries to change it by hand and ship it REPOSITORY R264 KApiDox REVISION DETAIL https://phabricator.kde.org/D28811 To: ognarb, cblack,

D28811: Add drawer on mobile and clean code

ochurlaud added a comment. Except these clarifications, I'm not against this change REPOSITORY R264 KApiDox REVISION DETAIL https://phabricator.kde.org/D28811 To: ognarb, cblack, ochurlaud Cc: kde-frameworks-devel, kde-doc-english, LeGast00n, cblack, gennad, fbampaloukas, michaelh,

D28811: Add drawer on mobile and clean code

ochurlaud requested changes to this revision. ochurlaud added a comment. This revision now requires changes to proceed. I really dislike how you close brackets in css : it's more dense and harder to read. You indent sometimes with 1,2 or 3 spaces: it should be consistent. Is it

D28734: Port to Aether Bootstrap 4

ochurlaud accepted this revision. ochurlaud added a comment. This revision is now accepted and ready to land. OK for me REPOSITORY R264 KApiDox BRANCH cblack+ognarb/bootstrap-4 (branched from master) REVISION DETAIL https://phabricator.kde.org/D28734 To: cblack, #vdg, ochurlaud Cc:

D28712: Redo api.kde.org to look more like Aether

ochurlaud added a comment. In D28712#645095 , @ognarb wrote: > Not tested yet but I like the style :) Though to be more consistent with the style used in other KDE websites, I would change the background color to #fff and use #f8f9fa for the

D28712: Redo api.kde.org to look more like Aether

ochurlaud accepted this revision. This revision is now accepted and ready to land. REPOSITORY R264 KApiDox BRANCH cblack/aether (branched from master) REVISION DETAIL https://phabricator.kde.org/D28712 To: cblack, #vdg, ognarb, ochurlaud Cc: ngraham, kde-frameworks-devel,

D28712: Redo api.kde.org to look more like Aether

ochurlaud added a comment. Nice work. This is a ship it for me! REPOSITORY R264 KApiDox REVISION DETAIL https://phabricator.kde.org/D28712 To: cblack, #vdg, ognarb, ochurlaud Cc: ngraham, kde-frameworks-devel, kde-doc-english, LeGast00n, cblack, gennad, fbampaloukas, GB_2, michaelh,

D26197: Display fully qualified class/namespace name as page header

ochurlaud added a comment. One minor correction then you can ship it INLINE COMMENTS > generator.py:456 > +else: > +mapping['classname'] = mapping['fullname'] = None > Prefer the use of 2 settings to None instead of this double one. It's more explicit for

D25632: Fix module imports with Python3

ochurlaud accepted this revision. ochurlaud added a comment. This revision is now accepted and ready to land. If it still works it's OK to push. REPOSITORY R264 KApiDox BRANCH bugfix/graphvizImportPython3 REVISION DETAIL https://phabricator.kde.org/D25632 To: cordlandwehr, ochurlaud

D24568: Provide clang-format target with a KDE Frameworks style file

ochurlaud added a comment. In D24568#546289 , @kossebau wrote: > There is also https://techbase.kde.org/Policies/Frameworks_Coding_Style which though missed the move from techbase to community, other than the other policies. > > I suspect

D22421: generate list of data files on the fly

ochurlaud accepted this revision. ochurlaud added a comment. This revision is now accepted and ready to land. Nice! Thx Harald! REPOSITORY R264 KApiDox BRANCH master REVISION DETAIL https://phabricator.kde.org/D22421 To: sitter, ochurlaud Cc: kde-frameworks-devel, kde-doc-english,

D14957: Create a first example of the use of KMessageBox

This revision was automatically updated to reflect the committed changes. Closed by commit R236:6748d24d5fb2: Create a first example of the use of KMessageBox (authored by ochurlaud). REPOSITORY R236 KWidgetsAddons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D14957?vs=40094=40095

D14957: Create a first example of the use of KMessageBox

ochurlaud marked an inline comment as done. REPOSITORY R236 KWidgetsAddons BRANCH master REVISION DETAIL https://phabricator.kde.org/D14957 To: ochurlaud, #frameworks, dfaure Cc: dfaure, kde-frameworks-devel, michaelh, ngraham, bruns

D14957: Create a first example of the use of KMessageBox

ochurlaud updated this revision to Diff 40094. ochurlaud added a comment. Fix as dfaure said REPOSITORY R236 KWidgetsAddons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D14957?vs=40092=40094 BRANCH master REVISION DETAIL https://phabricator.kde.org/D14957 AFFECTED FILES

D14955: Move example from techbase to own repo

This revision was automatically updated to reflect the committed changes. Closed by commit R311:87a84ac866c5: Move example from techbase to own repo (authored by ochurlaud). REPOSITORY R311 KWallet CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D14955?vs=40090=40093 REVISION DETAIL

D14957: Create a first example of the use of KMessageBox

ochurlaud updated this revision to Diff 40092. ochurlaud added a comment. Added licences and add some changes to fit what was done on D14955 REPOSITORY R236 KWidgetsAddons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D14957?vs=40082=40092

D14955: Move example from techbase to own repo

ochurlaud marked 4 inline comments as done. REPOSITORY R311 KWallet REVISION DETAIL https://phabricator.kde.org/D14955 To: ochurlaud, #frameworks, valir, dfaure Cc: dfaure, aacid, elvisangelaccio, kde-frameworks-devel, michaelh, ngraham, bruns

D14955: Move example from techbase to own repo

ochurlaud updated this revision to Diff 40090. ochurlaud added a comment. Added Licence, made it compile within the whole project and fix issues raised by dfaure. I think we shouldn't care about making the example self-standing. It's aimed at tutoring people to use the lib, not ECM nor

D14955: Move example from techbase to own repo

ochurlaud added a comment. In D14955#312279 , @aacid wrote: > -1 > > The example is not getting compiled it will rot I'm not sure to understand: it *can* be compiled, whereas on the current status (in the wiki) it will never be !

[Tutorials] Proposition of tutorial organization

Dear framework maintainers, Hidden somewhere on techbase.kde.org, one can find this: https://techbase.kde.org/Development/Tutorials Not everything is up-to-date nor at the right level. It's often too complex, not showing fully that Framework libs can be used alone (currently it seems that if

D14957: Create a first example of the use of KMessageBox

ochurlaud added a reviewer: Frameworks. REPOSITORY R236 KWidgetsAddons REVISION DETAIL https://phabricator.kde.org/D14957 To: ochurlaud, #frameworks Cc: kde-frameworks-devel, michaelh, ngraham, bruns

D14957: Create a first example of the use of KMessageBox

ochurlaud created this revision. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. ochurlaud requested review of this revision. REVISION SUMMARY Based on https://techbase.kde.org/Development/Tutorials/First_program TEST PLAN It compiles REPOSITORY R236

D14955: Move example from techbase to own repo

ochurlaud added reviewers: Frameworks, valir. REPOSITORY R311 KWallet REVISION DETAIL https://phabricator.kde.org/D14955 To: ochurlaud, #frameworks, valir Cc: kde-frameworks-devel, michaelh, ngraham, bruns

D14955: Move example from techbase to own repo

ochurlaud created this revision. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. ochurlaud requested review of this revision. REVISION SUMMARY Add example based on https://techbase.kde.org/Development/Tutorials/KWallet so that it can be kept up-to-date. The

D11330: Remove shebangs from non-executable files

ochurlaud accepted this revision. This revision is now accepted and ready to land. REPOSITORY R264 KApiDox BRANCH master REVISION DETAIL https://phabricator.kde.org/D11330 To: lbeltrame, #frameworks, ochurlaud Cc: #documentation, michaelh, ngraham, skadinna

D11315: Look first for qhelpgenerator-qt5 for help generation

ochurlaud accepted this revision. ochurlaud added a comment. This revision is now accepted and ready to land. Thx for this change REPOSITORY R264 KApiDox BRANCH master REVISION DETAIL https://phabricator.kde.org/D11315 To: lbeltrame, #frameworks, ochurlaud Cc: fvogt, #documentation,

D6604: add a metainfo.yaml to make ECM a proper framework

ochurlaud added a comment. At the time I thought about merging it but we would have to read document everything in doxygen , which would be completely different from what upstream cmake does. The ECM page is already linked from the left menu. Does it make sense to have it in

D6604: add a metainfo.yaml to make ECM a proper framework

ochurlaud added a comment. In https://phabricator.kde.org/D6604#123635, @sitter wrote: > related to https://phabricator.kde.org/D6603 > > @ochurlaud I hope this will not break api.kde, given ECM hasn't had a metainfo.yaml previously it seems to have some sort of special arrangement?

D5275: Kapidox generates qch files

ochurlaud added a comment. In https://phabricator.kde.org/D5275#99677, @antonanikin wrote: > Hi, Olivier. > > Your QCH files displayed normally but with one bug - broken links for some JavaScript code. For example open `KTipDialog` class doc from your `Frameworks` file. Then try to

D5275: Kapidox generates qch files

ochurlaud updated this revision to Diff 13209. ochurlaud added a comment. Corrected antonanikin issue REPOSITORY R264 KApiDox CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D5275?vs=13076=13209 BRANCH olivier/do_qch REVISION DETAIL https://phabricator.kde.org/D5275 AFFECTED

D5275: Kapidox generates qch files

ochurlaud updated this revision to Diff 13076. ochurlaud edited the summary of this revision. ochurlaud added a comment. Update: Order alphabetically the libs in a product REPOSITORY R264 KApiDox CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D5275?vs=13045=13076 BRANCH

D5275: Kapidox generates qch files

ochurlaud added a comment. Here 3 examples. Frameworks only contains KApiDox, KConfigWidgets and KItemModels. I could have generated all of them of course... F3526219: okular.qch F3526218: frameworks.qch

D5275: Kapidox generates qch files

ochurlaud added reviewers: mwolff, apol. ochurlaud added a comment. I add some people who may be able to tell me if the output can be used in the kdevelop generation (because I heard you use it). I don't use qch files myself, so I need an external view REPOSITORY R264 KApiDox

D5275: Kapidox generates qch files

ochurlaud edited the summary of this revision. ochurlaud added reviewers: kossebau, KDevelop. REPOSITORY R264 KApiDox REVISION DETAIL https://phabricator.kde.org/D5275 To: ochurlaud, kossebau, #kdevelop Cc: #frameworks, #documentation, skadinna

D5275: Kapidox generates qch files

ochurlaud created this revision. Restricted Application added projects: Frameworks, Documentation. Restricted Application added subscribers: Documentation, Frameworks. REPOSITORY R264 KApiDox BRANCH olivier/do_qch REVISION DETAIL https://phabricator.kde.org/D5275 AFFECTED FILES

Re: Phabricator: All repositories registered - upcoming workflow changes

Le dimanche 29 janvier 2017, 16:24:46 CET Luigi Toscano a écrit : > Il 29 gennaio 2017 16:16:51 CET, Olivier Churlaud <oliv...@churlaud.com> ha scritto: > >Le dimanche 29 janvier 2017, 08:32:21 CET Ben Cooksley a écrit : > >> We've just completed the registration of a

Re: Phabricator: All repositories registered - upcoming workflow changes

Le dimanche 29 janvier 2017, 08:32:21 CET Ben Cooksley a écrit : > Hi everyone, Hi, > We've just completed the registration of all mainline repositories > (not including Websites or Sysadmin namespaced ones) on Phabricator. > Thanks go to Luigi Toscano for providing significant assistance with >

Re: Review Request 129445: Add FreeBSD to metainfo.yaml.

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129445/#review101028 --- Ship it! Ship It! - Olivier Churlaud On Nov. 20, 2016

Re: Review Request 119723: Show q_properties at the top of class documentation

don't have anything against the change. +1 for me - Olivier Churlaud On Oct. 21, 2016, 1:14 p.m., David Edmundson wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.

Re: Review Request 128606: [kapidox] Read library fancyname from metainfo.yaml before parsing it from CMakeLists.txt

where the CMakeLists.txt is unavailable (like a sub-library). Thx for the fix! - Olivier Churlaud On Aug. 5, 2016, 1:34 a.m., Daniel Vrátil wrote: > > --- > This is an automatically generated e-mail. To reply, visit

Re: Review Request 120000: New Thermal and Electrical Units and Unit Convienience Function for KUnitConversion

g/r/12/#comment65872> Please add `@return Unit as a string` here - Olivier Churlaud On July 23, 2016, 11 a.m., Garret Wassermann wrote: > > --- > This is an automatically generated e-mail. To reply,

Re: Review Request 120000: New Thermal and Electrical Units and Unit Convienience Function for KUnitConversion

before friday, take it for a "Ship it" (and don't forget to add a line "REVIEW: 12" in your commit message) - Olivier Churlaud On July 23, 2016, 11 a.m., Garret Wassermann wrote: > > --- > This is an a

Re: Review Request 128476: Add scrollbars to kcm started without systemsettings5

2016, 4:20 p.m., Olivier Churlaud wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/128476/ > --- > &

Re: Review Request 128490: Checksums tab: replace groupbox with vertical separator

> On July 20, 2016, 3:48 p.m., Thomas Pfeiffer wrote: > > +1 > > It may still scare people, but I think it's a good compromise. If we get bad feedback , it can be reverted. But I think we won't - Olivier --- This is an automatically

Re: Review Request 128413: Add clipboard buttons in the Checksums tab

> On July 16, 2016, 12:30 p.m., Elvis Angelaccio wrote: > > Ping? > > > > (String freeze for frameworks is next Saturday, right?) I would say: replace the [copy] button by a select on click. and then ship it - Olivier --- This is an

Re: Review Request 128490: Checksums tab: replace groupbox with vertical separator

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128490/#review97664 --- Ship it! Ship It! - Olivier Churlaud On July 20, 2016

Re: Review Request 128466: Rename Checksums tab to Integrity

to calculate it without verifying > against another checksum) > - more explanatory text (the intro sentenced comes from Wikipedia) to > inform the user > > This is the result: > > ``` > Checksum > > > A checksum

Re: Review Request 120000: New Thermal and Electrical Units and Unit Convienience Function for KUnitConversion

see anything to say against the patch. Yet as I never worked on the project, I cannot give the ship it... src/unit.h (line 182) <https://git.reviewboard.kde.org/r/12/#comment65787> You should remove theses trailing spaces. - Olivier Churlaud On July 18, 2016, 11:55 p.m.,

Re: Review Request 128466: Rename Checksums tab to Integrity

to calculate it without verifying > against another checksum) > - more explanatory text (the intro sentenced comes from Wikipedia) to > inform the user > > This is the result: > > ``` > Checksum > > > A checksum

Re: Review Request 128466: Rename Checksums tab to Integrity

to calculate it without verifying > against another checksum) > - more explanatory text (the intro sentenced comes from Wikipedia) to > inform the user > > This is the result: > > ``` > Checksum > > > A checksum

Re: Review Request 128466: Rename Checksums tab to Integrity

to calculate it without verifying > against another checksum) > - more explanatory text (the intro sentenced comes from Wikipedia) to > inform the user > > This is the result: > > ``` > Checksum > > > A checksum

Re: Review Request 128466: Rename Checksums tab to Integrity

to calculate it without verifying > against another checksum) > - more explanatory text (the intro sentenced comes from Wikipedia) to > inform the user > > This is the result: > > ``` > Checksum > > > A checksum

Re: Review Request 128466: Rename Checksums tab to Integrity

to calculate it without verifying > against another checksum) > - more explanatory text (the intro sentenced comes from Wikipedia) to > inform the user > > This is the result: > > ``` > Checksum > > > A checksum

Re: Review Request 128466: Rename Checksums tab to Integrity

ed in > Rangar's comment in the review #128283) > > I do agree that the two work-flows are separate. Still, the UI does not > make that obvious to me. The UI does not make it obvious that it *has* two > work-flows. > > So, from my point of view, these are the p

Re: Review Request 128466: Rename Checksums tab to Integrity

> On July 18, 2016, 2:05 p.m., Sebastian Kügler wrote: > > Please don't ship it, yet. > > > > > > I find the UI illogical. There's a groupbox grouping the checksum buttons, > > but then you can input the checksum above, so essentially, the groupbox is > > unnecessary and confusing. > > > >

Review Request 128476: Add scrollbars to kcm started without systemsettings5

://git.reviewboard.kde.org/r/128476/diff/ Testing --- Compiles, behave as expected (expect for the size) Thanks, Olivier Churlaud ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 128464: Allow scrolling in config windows for small screens

marked as submitted. Review request for KDE Frameworks and Eike Hein. Changes --- Submitted with commit 574abcd05ece484da77635e2b8f23d3f4c1af5dd by Olivier CHURLAUD to branch master. Bugs: 362234 https://bugs.kde.org/show_bug.cgi?id=362234 Repository: kconfigwidgets Description

Re: Review Request 128464: Allow scrolling in config windows for small screens

83c96b6 Diff: https://git.reviewboard.kde.org/r/128464/diff/ Testing --- Compile and tested: Good behavior Thanks, Olivier Churlaud ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde

Re: Review Request 128464: Allow scrolling in config windows for small screens

. With this fix I can. The config page has now scroll bars when needed. Diffs (updated) - src/kconfigdialog.cpp 83c96b6 Diff: https://git.reviewboard.kde.org/r/128464/diff/ Testing --- Compile and tested: Good behavior Thanks, Olivier Churlaud

Re: Review Request 128464: Allow scrolling in config windows for small screens

appear too small. I don't know how to define a good default size... - Olivier Churlaud On July 16, 2016, 12:02 p.m., Olivier Churlaud wrote: > > --- > This is an automatically generated e-mail. To reply, visit

Re: Review Request 128464: Allow scrolling in config windows for small screens

it: https://git.reviewboard.kde.org/r/128464/#review97467 --- On July 16, 2016, 2:52 a.m., Olivier Churlaud wrote: > > --- > This is an automatically generated e-mail. To rep

Re: Review Request 128464: Allow scrolling in config windows for small screens

Churlaud On July 16, 2016, 2:52 a.m., Olivier Churlaud wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.

Re: QIcon::fromTheme() does not load scalable icons

From: Antonio Rojas <aro...@archlinux.org> To:kde-frameworks-devel@kde.org Subject: Re: QIcon::fromTheme() does not load scalable icons Message-ID:<nm2t7t$tlh$1...@ger.gmane.org> Content-Type: text/plain; charset="ISO-8859-1" Olivier Churlaud wrote: > >Can you

Re: QIcon::fromTheme() does not load scalable icons

the problem comes from upstream or packaging. Where should I open the bug? Cheers Olivier Cheers Elvis [1]: http://lists-archives.com/kde-devel/34061-how-to-properly-install-app-icons.html Thank you Olivier -- Olivier CHURLAUD Engineer Student at Ecole Centrale de Lyon in Dual Degree at TU

QIcon::fromTheme() does not load scalable icons

as it makes a lot of redundancy. Can you point me to the right place where to open the bug? Thank you Olivier -- Olivier CHURLAUD Engineer Student at Ecole Centrale de Lyon in Dual Degree at TU Berlin, M.Sc. Elektrotechnik @: oliv...@churlaud.com tel: +49 (0)1575-2931348 in: http://linkedin.com

Re: Review Request 128397: KIconEngine: Fix QIcon::hasThemeIcon always returning true

://sourceforge.net/p/texstudio/bugs/1837/ which was related to bug 365130. Not completely though, as the icon of the app is not shown in the taskbar (but it might be another bug). Looking forward for the release. - Olivier Churlaud On July 8, 2016, 2:24 p.m., David Rosca wrote

Re: Review Request 128397: KIconEngine: Fix QIcon::hasThemeIcon always returning true

/show_bug.cgi?id=365130 - Olivier Churlaud On July 8, 2016, 12:52 p.m., David Rosca wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.

Re: Review Request 128192: Remove man pages for scripts that no longer exist

. But I think it's right. Do you know how I should generate this for the new script? - Olivier Churlaud On June 15, 2016, 12:50 p.m., Michael Palimaka wrote: > > --- > This is an automatically generated e-mail. To reply, visit

Re: Replacing the "- name: All" platform wildcard

Le 15.06.2016 10:16, Andreas Cord-Landwehr a écrit : Hey all, in preparation for getting information about Android as supported platform on api.kde.org, I would like to purge the "- name: All" platform wildcard from all metainfo.yaml files and replace it by: - name: Linux - name:

KApiDox Maintainship

like some explanations about what being the maintainer means, what are the responsibilities and constrains, and of course, after I'll know that, if it's ok that I change the Alex's name with mine for Kapidox maintainer. Cheers Olivier -- Olivier CHURLAUD Engineer Student at Ecole Centrale de

Re: Review Request 128008: Correct documentation issues

marked as submitted. Review request for KDE Frameworks and Vishesh Handa. Changes --- Submitted with commit 74712f28651b72f160367750e7c46f0c7674f051 by Olivier CHURLAUD to branch master. Repository: baloo Description --- The md-files were not parsed. Now they are. Diffs

Review Request 128008: Correct documentation issues

://git.reviewboard.kde.org/r/128008/diff/ Testing --- Tested on kapidox. Thanks, Olivier Churlaud ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 127747: Create a new script that generate the documentation for all projects following the syntax I proposed

marked as submitted. Review request for KDE Frameworks, Albert Astals Cid, Alex Merry, Aurélien Gâteau, and Allen Winter. Changes --- Submitted with commit b9bba891f8962b6af79d20edb7216577f54cacbd by Olivier CHURLAUD to branch master. Repository: kapidox Description --- Keep

Re: Review Request 127747: Create a new script that generate the documentation for all projects following the syntax I proposed

here > > work needed by others (sysadmin, EBN, etc.) to make apidox generation work > > with your patch committed? What's the action KDE developers must take after > > this, if any? > > Olivier Churlaud wrote: > Thank you for the review Michel. The only need will be t

Re: Review Request 127747: Create a new script that generate the documentation for all projects following the syntax I proposed

here > > work needed by others (sysadmin, EBN, etc.) to make apidox generation work > > with your patch committed? What's the action KDE developers must take after > > this, if any? > > Olivier Churlaud wrote: > Thank you for the review Michel. The only need will be t

Re: Review Request 127747: Create a new script that generate the documentation for all projects following the syntax I proposed

here > > work needed by others (sysadmin, EBN, etc.) to make apidox generation work > > with your patch committed? What's the action KDE developers must take after > > this, if any? > > Olivier Churlaud wrote: > Thank you for the review Michel. The only need will be t

Re: Review Request 127747: Create a new script that generate the documentation for all projects following the syntax I proposed

---------- On April 25, 2016, 11:49 p.m., Olivier Churlaud wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/127747/ > ---

Re: Review Request 127747: Create a new script that generate the documentation for all projects following the syntax I proposed

> On April 28, 2016, 10:02 a.m., Olivier Churlaud wrote: > > Any other feedback? What should I do? > > Allen Winter wrote: > I suppose I need to find the time to test this out on the actual > api.kde.org machine. can I run it on the old KDE4 kdelibs KDE/4.14 branch?

Re: Review Request 127747: Create a new script that generate the documentation for all projects following the syntax I proposed

> On April 28, 2016, 10:02 a.m., Olivier Churlaud wrote: > > Any other feedback? What should I do? > > Allen Winter wrote: > I suppose I need to find the time to test this out on the actual > api.kde.org machine. can I run it on the old KDE4 kdelibs KDE/4.14 branch?

Re: Review Request 127747: Create a new script that generate the documentation for all projects following the syntax I proposed

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127747/#review95148 --- Nothing? What should I do? - Olivier Churlaud On avr. 25

Re: Review Request 127747: Create a new script that generate the documentation for all projects following the syntax I proposed

> On April 28, 2016, 10:02 a.m., Olivier Churlaud wrote: > > Any other feedback? What should I do? > > Allen Winter wrote: > I suppose I need to find the time to test this out on the actual > api.kde.org machine. can I run it on the old KDE4 kdelibs KDE/4.14 branch?

Re: Review Request 127747: Create a new script that generate the documentation for all projects following the syntax I proposed

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127747/#review94943 --- Any other feedback? What should I do? - Olivier Churlaud

Re: Review Request 127747: Create a new script that generate the documentation for all projects following the syntax I proposed

> On April 26, 2016, 1:16 a.m., Albert Astals Cid wrote: > > metainfo.yaml, line 13 > > <https://git.reviewboard.kde.org/r/127747/diff/2/?file=460336#file460336line13> > > > > This is already in line 3, doesn't seem like a good idea to duplicate > >

Re: Review Request 127747: Create a new script that generate the documentation for all projects following the syntax I proposed

rks, which will be more difficult to maintain. - Olivier --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127747/#review94845 --------

Re: Review Request 127747: Create a new script that generate the documentation for all projects following the syntax I proposed

cases. File Attachments This is an example of what I generated. (Threadweaver is duplicated and modified to test different scenarios) https://git.reviewboard.kde.org/media/uploaded/files/2016/04/25/2e4549e4-7c17-416c-9a72-b82d3bba18b3__doc.tar.gz Thanks, Olivier Churlaud