Re: [Geotools-devel] Promoting gt-ysld to a supported module

2017-01-24 Thread Jody Garnett
I am not as keen as Martin was, I expect that Optionals will take the place of the idea of null objects long term. What I am keen on is making our developer guide not suck; so if we are not being strict about null we should relax the guide (citing returning a null collection). I am kind of okay wi

Re: [Geotools-devel] [Geoserver-devel] GeoTools / GeoServer Meeting 2017-01-24

2017-01-24 Thread Torben Barsballe
I've looked into the Cobertura Tests: Seems to work fine on GeoTools, confirmed it is working on classes that contain Java 8 syntax like lambda expressions. Looks like GeoServer is currently using a different (earlier) version of Cobertura that produces copious errors. After updating the Cobertura

[Geotools-devel] [JIRA] (GEOT-5622) YSLD Validation for aliases

2017-01-24 Thread Torben Barsballe (JIRA)
Title: Message Title Torben Barsballe crea

[Geotools-devel] Code freeze for 17-beta release: 18 February

2017-01-24 Thread Ben Caradoc-Davies
The 17-beta release is scheduled for 18 February, so plan to get any commits you want in 17.0 merged on master before this date. Only bug fixes will be accepted on master during the code freeze, which lasts from the 17-beta release until the 17-RC release and the creation of the 17.x branch. I

[Geotools-devel] GeoTools / GeoServer Meeting 2017-01-24

2017-01-24 Thread Ben Caradoc-Davies
** *GeoTools / GeoServer Meeting 2017-01-24* * Attending Ben Caradoc-Davies Jody Garnett Torben Barsballe Mike Pumphrey Jukka Rahkonen Kevin Smith Reggie Beckwith Agenda * Sprint Planning / Update * Code Freeze Planning * YSLD Promotion from Unsupported to Extensio

Re: [Geotools-devel] Promoting gt-ysld to a supported module

2017-01-24 Thread Matt Kruszewski
Thanks Andrea -- agreed. I tried to note that returning null actually seemed valid in most of these particular cases, but I didn't know how strictly the "rule" (from the developer guide) is enforced. On Tue, Jan 24, 2017 at 12:57 PM, Torben Barsballe < tbarsba...@boundlessgeo.com> wrote: > I agre

Re: [Geotools-devel] Promoting gt-ysld to a supported module

2017-01-24 Thread Torben Barsballe
I agree that strictly not returning null ever is overkill. Given that, we should probably update the dev guide: http://docs.geotools.org/latest/developer/conventions/code/null.html Torben On Tue, Jan 24, 2017 at 10:11 AM, Andrea Aime wrote: > On Tue, Jan 24, 2017 at 6:57 PM, Matt Kruszewski < >

Re: [Geotools-devel] Promoting gt-ysld to a supported module

2017-01-24 Thread Andrea Aime
On Tue, Jan 24, 2017 at 6:57 PM, Matt Kruszewski < mkruszew...@boundlessgeo.com> wrote: > Hello everyone, > > I've started applying the developer guide standards to the gt-ysld module, > keeping notes in this gist: > > https://gist.github.com/mattkrusz/b010eb29aa784025a639c473b768801e > I don't a

Re: [Geotools-devel] Promoting gt-ysld to a supported module

2017-01-24 Thread Matt Kruszewski
Hello everyone, I've started applying the developer guide standards to the gt-ysld module, keeping notes in this gist: https://gist.github.com/mattkrusz/b010eb29aa784025a639c473b768801e Please take a look and provide any perspective you have on the current design -- things that definitely need t