> On Jan. 17, 2014, 10:46 a.m., Sebastian Kügler wrote:
> > Hi Scarlett,
> > 
> > Congrats to your first patch (that I see coming by)! I think it's a useful 
> > contribution and as far as I can judge, almost good to go in. I've put a 
> > comment inline how the explanation could be further improved.
> > 
> > I don't know much about manpages, so I'll let someone else comment on the 
> > mechanics.
> > 
> > Thanks for taking care of this.

Sebas, you are month too late with "Congrats to your first patch", Scarlett is 
the kdepim doc maintainer since more than a half year.

@Scarlett: will review later, thanks for waiting


- Burkhard


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/115063/#review47561
-----------------------------------------------------------


On Jan. 17, 2014, 2:52 a.m., Scarlett Clark wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/115063/
> -----------------------------------------------------------
> 
> (Updated Jan. 17, 2014, 2:52 a.m.)
> 
> 
> Review request for Documentation, KDE Frameworks, Jonathan Riddell, and 
> Sebastian Kügler.
> 
> 
> Repository: kservice
> 
> 
> Description
> -------
> 
> While building the package for kubuntu lintian complained about missing 
> manpage for binary. So I created one :) This is my first manpage so be kind. 
> I have no idea how to determine what type eg 1, 7, 8.
> 
> 
> Diffs
> -----
> 
>   docs/CMakeLists.txt 86ed60b 
>   docs/desktoptojson/CMakeLists.txt PRE-CREATION 
>   docs/desktoptojson/man-desktoptojson.8.docbook PRE-CREATION 
> 
> Diff: https://git.reviewboard.kde.org/r/115063/diff/
> 
> 
> Testing
> -------
> 
>  meinproc5 --stylesheet 
> /usr/share/ksgmltools2/customization/kde-include-man.xsl --check 
> man-desktoptojson.8.docbook 
> 
> 
> Thanks,
> 
> Scarlett Clark
> 
>

_______________________________________________
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Reply via email to