kicad-developers team mailing list archive
-
kicad-developers team
-
Mailing list archive
-
Message #36128
Re: [PATCH] Python FP wizard helper: docstrings and rounded/chamfered rects
-
To:
kicad-developers@xxxxxxxxxxxxxxxxxxx
-
From:
Wayne Stambaugh <stambaughw@xxxxxxxxx>
-
Date:
Tue, 12 Jun 2018 09:07:42 -0400
-
In-reply-to:
<CAG1r56+Fi95xr+S2wDsNJT8d3PuXKDRSsM=8yVTxBVB1uu0cUw@mail.gmail.com>
-
User-agent:
Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.8.0
Hey John,
I like the idea of using doxygen to document the python plugins. The
current Doxyfile does not include .py files so that would need to
change. Before we do that, I would like to see a new section (maybe
"Python Plugins") added to the documentation to separate the python
plugin code from the c++ source documentation. I can commit your patch
as is and you can make the doxygen changes in a later patch or I can
wait for you to create a new patch with all of the changes. I'm fine
either way.
Cheers,
Wayne
On 6/4/2018 7:33 AM, John Beard wrote:
> Hi,
>
> Here is a simple patch sequence for the Python Footprint Wizard helpers:
>
> 1) Minor spelling and formatting tidy-up
> 2) Add docstrings for the wizard base. As this is intended to be used
> by writers of new plugins, having the functions documented is probably
> a Good Idea (TM)
> 3) Add rounded rectangle and chamfered rectangle helpers. Useful for
> some footprints or even board outlines.
>
> I used Doxygen-style docstrings, but I haven't actually done anything
> about building actual output docs with it. Any thoughts of if that
> should be done, and if so, where to put it?
>
> There shouldn't be anything here that will break existing plugins, the
> only API changes are additions.
>
> Cheers,
>
> John
>
>
>
> _______________________________________________
> Mailing list: https://launchpad.net/~kicad-developers
> Post to : kicad-developers@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~kicad-developers
> More help : https://help.launchpad.net/ListHelp
>
Follow ups
References