Simon, Please be patient. I know things have been a bit slow on the mailing list recently. This is not that unusual in the summer as developers are on vacation or busy doing other things.
I took a look at your document and have the following comments: Avoid referring to the Python plugin as a new feature. This will eliminate the need to go back and change the document when it no longer is a new feature. The Python plugin support has been around for a few years. Only the action plugin in support is new. The title should read "Python Plugin Development for Pcbnew" using title capitalization as shown. It's common to use title capitalization for all section titles. All the section titles in your document use sentence capitalization. "Pcbnew" should always be capitalized since it is a proper noun and is the official name. PEP 8 is the Python coding standard. This doesn't seem like it's related to Python packaging. Are you sure this is the correct PEP? It might not be a bad idea to let developers know that additional paths can be added to the Python runtime environment when designing complex plugins using the sys.path or the equally handy PYTHONPATH environment variable. This handy Python feature is often overlooked by developers new to Python. Your patch does not add the new document to the Doxygen configuration file so it will not get added to to the developers documentation. Would someone with more knowledge about the Pcbnew Python plugin support verify that this information is correct? It looks fine to me but I'm not completely familiar with the new action plugin support. Cheers, Wayne On 6/17/2017 5:59 AM, Simon Küppers wrote: > Since there hasn't been any followup, attached is the patch. > > Best Regards! > > > _______________________________________________ > Mailing list: https://launchpad.net/~kicad-developers > Post to : kicad-developers@lists.launchpad.net > Unsubscribe : https://launchpad.net/~kicad-developers > More help : https://help.launchpad.net/ListHelp > _______________________________________________ Mailing list: https://launchpad.net/~kicad-developers Post to : kicad-developers@lists.launchpad.net Unsubscribe : https://launchpad.net/~kicad-developers More help : https://help.launchpad.net/ListHelp