Review Request 115185: Integrate kf5dot

Aurélien Gâteau agateau at kde.org
Wed Jan 22 07:48:03 UTC 2014



> On Jan. 21, 2014, 6:16 p.m., Alex Merry wrote:
> > - files should have licenses and encoding marks
> > - "kf5dot" is not very descriptive as a name.  I would call the module "dependencies" or "depdiags" or something like that, and the scripts something similar but with a k or kf5 prefix
> > - it would be nice if kgenframeworksapidox could do all the magic itself, when suitable option(s) are passed (although that can be something to implement later)

> - files should have licenses and encoding marks

Will fix.

> - "kf5dot" is not very descriptive as a name.  I would call the module "dependencies" or "depdiags" or something like that, and the scripts something similar but with a k or kf5 prefix

I am not happy with the name either, but I think this can be fixed in a different commit.

> - it would be nice if kgenframeworksapidox could do all the magic itself, when suitable option(s) are passed (although that can be something to implement later)

Agreed.


- Aurélien


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


On Jan. 21, 2014, 5:58 p.m., Aurélien Gâteau wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/115185/
> -----------------------------------------------------------
> 
> (Updated Jan. 21, 2014, 5:58 p.m.)
> 
> 
> Review request for KDE Frameworks and Alex Merry.
> 
> 
> Repository: kapidox
> 
> 
> Description
> -------
> 
> This big patch includes all of the kf5dot repository inside kapidox. Python code is in src/kapidox/kf5dot, scripts are in src/.
> 
> I replayed all the commits from the kf5dot repository within the kapidox repository using `git am` to avoid loosing history. As such, I plan to do the merge using --no-ff.
> 
> 
> Diffs
> -----
> 
>   src/kapidox/kf5dot/frameworkdb.py PRE-CREATION 
>   src/kf5dot-generate PRE-CREATION 
>   src/kf5dot-generate-all PRE-CREATION 
>   src/kf5dot-prepare PRE-CREATION 
>   README.md 660e9c3 
>   src/kapidox/kf5dot/framework.py PRE-CREATION 
>   setup.py 025afdb 
>   src/kapidox/kf5dot/.gitignore PRE-CREATION 
>   src/kapidox/kf5dot/block.py PRE-CREATION 
> 
> Diff: https://git.reviewboard.kde.org/r/115185/diff/
> 
> 
> Testing
> -------
> 
> Generated all diagrams. Works as expected.
> 
> 
> Thanks,
> 
> Aurélien Gâteau
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140122/b1e3b11a/attachment.html>


More information about the Kde-frameworks-devel mailing list