On Sun, Nov 25, 2012 at 11:20 PM, Justin Deoliveira <[email protected]>wrote:
> A big +1 for this. A README in the root of repo will go a far way i think.
> Some links for it that could be relevant (at least on the GeoServer side):
>
> http://docs.geoserver.org/latest/en/developer/policies/patches.html (even
> has a section on pull requests)
> http://docs.geoserver.org/latest/en/developer/policies/comitting.html
> http://docs.geoserver.org/latest/en/developer/source.html
>
> What else do we want to include? Happy to help write the README once we
> figure out what we want to convey.
>
>
I am under the impression none of the documents above convey two important
bits:
- the need to discuss on the mailing list any non trivial change, and in
particular any API changes
- the presence of tests for changes
Cheers
Andrea
--
==
Our support, Your Success! Visit http://opensdi.geo-solutions.it for more
information.
==
Ing. Andrea Aime
@geowolf
Technical Lead
GeoSolutions S.A.S.
Via Poggio alle Viti 1187
55054 Massarosa (LU)
Italy
phone: +39 0584 962313
fax: +39 0584 1660272
mob: +39 339 8844549
http://www.geo-solutions.it
http://twitter.com/geosolutions_it
-------------------------------------------------------
------------------------------------------------------------------------------
Monitor your physical, virtual and cloud infrastructure from a single
web console. Get in-depth insight into apps, servers, databases, vmware,
SAP, cloud infrastructure, etc. Download 30-day Free Trial.
Pricing starts from $795 for 25 servers or applications!
http://p.sf.net/sfu/zoho_dev2dev_nov
_______________________________________________
GeoTools-Devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geotools-devel
