[MetaCRS] Give Calvin Metcalf commit access to proj4js; proj4js review policy

[Andreas Hocevar]

Welcome Calvin, with four +1 votes and no objections you are now a proj4js
committer. Keep up the great work!

With a vote of three +1, proj4js now has a review policy - no merging of
pull requests without a peer review.

Thanks Hugues for the RFC suggestion. Although we do not have such a
process in place, I would ask everyone who has big changes in mind to
discuss them beforehand. It is my impression that this has been a part of
the project's culture anyway, and has happened with all big changes so far.

Andreas.


On Fri, Sep 6, 2013 at 9:48 AM, Hugues Wisniewski <
hugues.wisniewski at autodesk.com> wrote:

>  +1****
>
> ** **
>
> > Now that we're at least 3 active proj4js committers, I'd like to
> establish a review policy****
>
> +1 for that****
>
> ** **
>
> Something to possibly also consider is the concept of RFCs used on
> MapGuide and CsMap (Request For Change)****
>
> When many from different teams and companies work on the same project,
> it’s nice to be notified ahead of time of major changes coming up. Others
> have time to prepare and react, possibly asking or suggesting changes.****
>
> This is the CsMap page, which is based on the MapGuide RFC process****
>
> http://trac.osgeo.org/csmap/wiki/CsMapRfcs****
>
> ** **
>
> Hugues****
>
> ** **
>
> *From:* metacrs-bounces at lists.osgeo.org [mailto:
> metacrs-bounces at lists.osgeo.org] *On Behalf Of *Andreas Hocevar
> *Sent:* Thursday, September 05, 2013 8:36 PM
> *To:* metacrs at lists.osgeo.org; Calvin Metcalf; Mike Adair
> *Subject:* [MetaCRS] Give Calvin Metcalf commit access to proj4js;
> proj4js review policy****
>
> ** **
>
> Hey,****
>
> ** **
>
> I'd like to propose to give Calvin Metcalf commit access to the proj4js
> repository on github. Calvin has shown great talent bringing bleeding edge
> JavaScript technologies to proj4js, and the many pull request of his that I
> merged were all of outstanding quality.****
>
> ** **
>
> I am +1.****
>
> ** **
>
> Now that we're at least 3 active proj4js committers, I'd like to establish
> a review policy: Pull request - peer review - merge. This way every change
> will have a second pair of eyes look at it, which is good for QA.****
>
> ** **
>
> I am +1.
> ****
>
> ** **
>
> Thanks,****
>
> Andreas.****
>
>
> ****
>
> ** **
>
> --
> Andreas Hocevar
> OpenGeo - http://opengeo.org/
> Expert service straight from the developers.
>
> --
> Andreas Hocevar
> OpenGeo - http://opengeo.org/
> Expert service straight from the developers.
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osgeo.org/pipermail/metacrs/attachments/20130907/fe6e78c3/attachment.html>