On Mon, Jan 13, 2020 at 8:22 AM Jody Garnett <jody.garn...@gmail.com> wrote:
> I would like to propose MBStyle for graduation: > > Gold Star Plugin QA Test > <https://docs.geotools.org/latest/developer/procedures/check.html>: > > - Module maintainer: Jody Garnett (GeoCat) > > So far I asked Torben to review the PRs, should I route the PR review requests to you then? > > - Passed IP Check: Code was initially written by Boundless under > corporate CLA as a community module (indeed there is a proposal page > <https://github.com/geotools/geotools/wiki/Mapbox-Style> for the > community module) > - Releasable: there are no blocking bugs in mbstyle Jira > > <https://osgeo-org.atlassian.net/issues/?jql=project%20%3D%20GEOT%20AND%20component%20%3D%20mbstyle> > - Used in anger: shipped as part of boundless suite > > Hum... isn't the suite dead? Did it get some exposure? I have found a significant number of issues trying to translate the freely available OpenMapStyles styles (including hard crashes during the translation), see here, the first 20 or so rows: https://osgeo-org.atlassian.net/issues/?jql=text%20~%20mbstyles%20ORDER%20BY%20created%20DESC That said, many of these are now closed, with a few PRs more and it will more or less translate decently the osm-bright-gl OMT <https://github.com/openmaptiles/osm-bright-gl-style> style. The other OMT styles <https://github.com/openmaptiles?utf8=%E2%9C%93&q=gl-style&type=&language=> need some more work, if you want to play with them easily see this GeoServer data dir: https://github.com/geosolutions-it/openmaptiles (most of the issue now can be spotted by looking at the maps side by side with a Maputnik editor <https://maputnik.github.io/editor/#1/0/0>). > - Optimized: non-performance critical code > - Supported: community module documentation > > <https://docs.geotools.org/latest/userguide/unsupported/mbstyle/index.html> > has > required code example, and a copy of the mapbox style specification > indicating which aspects are implemented. I would like > > Additional: > > - Developers guide: code has been consistently reviewed during > development, subject to automated formatting and Andrea's recent QA safety > measures > - Code Coverage: Getting "Skipping JaCoCo execution due to missing > execution data file" any tips welcome > > IntelliJ reports a 87% coverage on the module, which is pretty good. (all of the issues I worked on either lacked a test or were testing for non compliant behavior). Cheers Andrea == GeoServer Professional Services from the experts! Visit http://goo.gl/it488V for more information. == Ing. Andrea Aime @geowolf Technical Lead GeoSolutions S.A.S. Via di Montramito 3/A 55054 Massarosa (LU) phone: +39 0584 962313 fax: +39 0584 1660272 mob: +39 339 8844549 http://www.geo-solutions.it http://twitter.com/geosolutions_it ------------------------------------------------------- *Con riferimento alla normativa sul trattamento dei dati personali (Reg. UE 2016/679 - Regolamento generale sulla protezione dei dati “GDPR”), si precisa che ogni circostanza inerente alla presente email (il suo contenuto, gli eventuali allegati, etc.) è un dato la cui conoscenza è riservata al/i solo/i destinatario/i indicati dallo scrivente. Se il messaggio Le è giunto per errore, è tenuta/o a cancellarlo, ogni altra operazione è illecita. Le sarei comunque grato se potesse darmene notizia. This email is intended only for the person or entity to which it is addressed and may contain information that is privileged, confidential or otherwise protected from disclosure. We remind that - as provided by European Regulation 2016/679 “GDPR” - copying, dissemination or use of this e-mail or the information herein by anyone other than the intended recipient is prohibited. If you have received this email by mistake, please notify us immediately by telephone or e-mail.*
_______________________________________________ GeoTools-Devel mailing list GeoTools-Devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/geotools-devel
