<p dir="ltr"><br>
On 26 Jan 2016 07:57, "David Adler" <<a href="mailto:dadler@adtechgeospatial.com">dadler@adtechgeospatial.com</a>> wrote:<br>
><br>
> We would like to add a DB2 provider to QGIS so that QGIS users can access spatial tables on DB2 for LUW and DB2 for z/OS.<br>
><br>
> We have implemented and tested much of this based on an existing database provider.<br>
><br>
> What is the mechanism to have this reviewed and approved?</p>
<p dir="ltr">Open a pull request on github, and let the automated CI run its checks over the code.</p>
<p dir="ltr">Tips to get things reviewed/merged faster:<br>
- Make sure all your API additions are correctly documented<br>
- Include unit tests. At a minimum a new vector provider needs a test which inherits from ProviderTestCase. See tests/src/python/test_provider_mssql.py for an example. This ensures that the provider behaves consistently with what QGIS expects from a vector data provider.</p>
<p dir="ltr">Nyall</p>
<p dir="ltr">><br>
><br>
><br>
> ---<br>
> This email has been checked for viruses by Avast antivirus software.<br>
> <a href="https://www.avast.com/antivirus">https://www.avast.com/antivirus</a><br>
><br>
> _______________________________________________<br>
> Qgis-developer mailing list<br>
> <a href="mailto:Qgis-developer@lists.osgeo.org">Qgis-developer@lists.osgeo.org</a><br>
> List info: <a href="http://lists.osgeo.org/mailman/listinfo/qgis-developer">http://lists.osgeo.org/mailman/listinfo/qgis-developer</a><br>
> Unsubscribe: <a href="http://lists.osgeo.org/mailman/listinfo/qgis-developer">http://lists.osgeo.org/mailman/listinfo/qgis-developer</a></p>