Thanks Niels. I'll take a look later. -----Original Message----- From: Niels Charlier [mailto:ni...@scitus.be] Sent: Wednesday, 19 December 2012 8:31 AM To: Caradoc-Davies, Ben (CESRE, Kensington) Cc: Jody Garnett; geotools-devel@lists.sourceforge.net; Angreani, Rini (CESRE, Kensington) Subject: Re: [Geotools-devel] proposal: Separate general complex feature classes from gt-app-schema
On 12/18/2012 04:13 AM, Ben Caradoc-Davies wrote: > On 17/12/12 18:58, Niels Charlier wrote: >> Cool, with no more objections and a +1 from the module maintainer and >> a few others I guess I can proceed? > > That depends what you mean by proceed: +1 is for the overarching API > change proposal (which classes are in which modules). In my view, you > should also seek approval from module maintainers for the final > version before it is committed; this is likely to be quick as the main > points of contention have been discussed. You can submit patches or > github pull requests, which are also becoming popular. > > Thanks for progressing this work, Niels. It is a much-needed improvement. > > Kind regards, > A pull request has been sent: https://github.com/geotools/geotools/pull/88 I assigned Rini, I guess she will be reviewing it? I also need a review for the changes in gt-main, I suppose Jody? Thanks for everyone who has with helped with this, and thanks in advance for the reviews! Kind Regards Niels ------------------------------------------------------------------------------ LogMeIn Rescue: Anywhere, Anytime Remote support for IT. Free Trial Remotely access PCs and mobile devices and provide instant support Improve your efficiency, and focus on delivering more value-add services Discover what IT Professionals Know. Rescue delivers http://p.sf.net/sfu/logmein_12329d2d _______________________________________________ GeoTools-Devel mailing list GeoTools-Devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/geotools-devel
