<div dir="ltr"><div><div>Hi Paolo,<br><br></div>There is a "How to add plugins to this repository" title in the "about plugins" page. I realized that this includes some of those notes you added at the previous post. May be we can extend these items.<br>
<br></div>Regards,<br>Necip.<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On 8 May 2014 09:59, Paolo Cavallini <span dir="ltr"><<a href="mailto:cavallini@faunalia.it" target="_blank">cavallini@faunalia.it</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi all.<br>
If someone points out the best place to write it, I'll add up these notes.<br>
Maybe dircetly on the HP <a href="http://plugins.qgis.org/" target="_blank">http://plugins.qgis.org/</a> ?<br>
All the best.<br>
<div class=""><br>
Il 30/04/2014 14:06, Vincent Picavet ha scritto:<br>
</div><div><div class="h5">> Hello,<br>
><br>
>> I am adding my notes on the plugins to be approved directly on the<br>
>> django interface, using the About field. I encourage others to do the<br>
>> same. Comments have to be removed before publication.<br>
><br>
> As you already said, it would be great if such a reviewer comment feature<br>
> would be available directly in the django interface.<br>
><br>
>> It would be useful to have a list of items to be checked for approval.<br>
><br>
> Some more items for code reviewing :<br>
> * english language used in code (comments, identifiers)<br>
> * good shape of source repository :<br>
> - no generated file in repository (ui_*.py, resources_rc.py, gen. help files…)<br>
> - good code organization (subfolders)<br>
> - code comments<br>
> - PEP8 & Python/QGIS guidelines compliance<br>
> - a README file<br>
> - a LICENCE file (GPL2/GPL2+ mandatory)<br>
> * Licence compliancy wrt external dependencies (e.g. no import of commercial<br>
> python module)<br>
> * "No evil" : track the obvious "rm -Rf ~/" code<br>
><br>
> As for external dependencies, I think it should be clearly stated somewhere if<br>
> the plugin has a need for non-free external dependency.<br>
> If some dependencies are not available in OSGeo4w Python, a clear indication<br>
> of how to install them should be given.<br>
><br>
> Great review work from your part Paolo btw !<br>
> We will try to join the effort soon.<br>
><br>
> Vincent<br>
><br>
>> Here a few items:<br>
>> * no .pyc files (correct?)<br>
>> * virus scan (how to do it? could this be automatic on the server side?)<br>
>> * if special data are needed to test the functioning, a small sample<br>
>> should be added<br>
>> * the plugin should go to the appropriate menu (Vector, Raster, Web,<br>
>> Database).<br>
>> I'm also checking for:<br>
>> * the presence of bugtracker and code repo<br>
>> * correctness of links<br>
>> * installation and basic functioning.<br>
>> Anyone wants to add?<br>
>> The good news is that we now have only 9 unapproved plugins:<br>
>> * 5 are being worked on<br>
>> * for 2, authors are not responding<br>
>> * 1 has issues with licence<br>
>> * 1 is considered unsuitable by the author.<br>
>> All the best.<br>
<br>
<br>
</div></div><div class="">--<br>
Paolo Cavallini - <a href="http://www.faunalia.eu" target="_blank">www.faunalia.eu</a><br>
</div>Corsi QGIS e PostGIS: <a href="http://www.faunalia.eu/training.html" target="_blank">http://www.faunalia.eu/training.html</a><br>
<div class="HOEnZb"><div class="h5">_______________________________________________<br>
Qgis-developer mailing list<br>
<a href="mailto:Qgis-developer@lists.osgeo.org">Qgis-developer@lists.osgeo.org</a><br>
<a href="http://lists.osgeo.org/mailman/listinfo/qgis-developer" target="_blank">http://lists.osgeo.org/mailman/listinfo/qgis-developer</a></div></div></blockquote></div><br></div>