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 to be changed, that may be OK as-is,
or that I may have missed.
Thanks,
Matt
On Mon, Jan 23, 2017 at 4:40 PM, Torben Barsballe <
tbarsba...@boundlessgeo.com> wrote:
> I have performed an IP review of the gt-ysld codebase. Results here:
> https://github.com/geotools/geotools/pull/1438
>
> This is my first time actually doing a proper IP review, so a reveiw would
> be appreciated before I merge it into the codebase.
>
> Thanks,
>
> Torben
>
> On Fri, Jan 20, 2017 at 3:13 PM, Torben Barsballe <
> tbarsba...@boundlessgeo.com> wrote:
>
>> I've added some documentation to the user manual (mostly pulled from the
>> YSLD README.md). YSLD Documentation PR here: https://github.com/geoto
>> ols/geotools/pull/1437
>>
>> Justin - thanks for the response.
>>
>> Torben
>>
>> On Fri, Jan 20, 2017 at 1:44 PM, Justin Deoliveira <jdeol...@gmail.com>
>> wrote:
>>
>>> Hey Torben,
>>>
>>> Thanks for pushing this forward. I am happy to keep my name on as module
>>> maintainer list as long as others see fit. I know I haven’t been the most
>>> active maintainer over the past while but unfortunately my involvement is
>>> driven by requirements from project work. That said if someone else is keen
>>> to maintain I will gladly step aside.
>>>
>>> -Justin
>>>
>>> On Fri, Jan 20, 2017 at 1:12 PM Torben Barsballe <
>>> tbarsba...@boundlessgeo.com> wrote:
>>>
>>>> Hello All,
>>>>
>>>> I am hoping to promote gt-ysld to a supported module.
>>>>
>>>> There are still a number of things that need to be done before this
>>>> module is ready to be promoted; I will be tackling these over the next few
>>>> weeks, with the aim of having gt-ysld be a supported module for GeoTools
>>>> 16.2 next month.
>>>>
>>>> - Add proper YSLD documentation to the GeoTools User Guide
>>>> - gt-ysld depends on gt-process. We need to either remove the
>>>> dependency on gt-process, or promote gt-process to a supported module.
>>>> - Perform an IP-check on the gt-ysld module.
>>>>
>>>>
>>>> Justin and Kevin - You are currently marked as the module maintainers
>>>> for the YSLD module. Are either or both of you willing to continue this
>>>> commitment when YSLD moves to a supported module?
>>>>
>>>> Torben
>>>>
>>>
>>
>
> ------------------------------------------------------------
> ------------------
> Check out the vibrant tech community on one of the world's most
> engaging tech sites, SlashDot.org! http://sdm.link/slashdot
> _______________________________________________
> GeoTools-Devel mailing list
> GeoTools-Devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/geotools-devel
>
>
------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, SlashDot.org! http://sdm.link/slashdot
_______________________________________________
GeoTools-Devel mailing list
GeoTools-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel
