[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: r8305 - branches/1.0-stabilization

From: <kfogel_at_collab.net>
Date: 2004-01-15 17:21:02 CET

kfogel@tigris.org writes:
> Log:
> * STATUS: Note new approval system for bindings and tools.
>
> [...]
>
> +There is a somewhat looser system for the bindings and tools areas,
> +because they are not core code, and because there are usually fewer
> +experts available to review changes there. A change to bindings or
> +tools can go in with a +1 from a full committer or a partial committer
> +for that area, at least one "+0" or "concept +1" from any other
> +committer, and no vetoes. We trust people to use their judgement and
> +not review changes if they don't have some competence to do so. The
> +goal is to ensure there are at least two pairs of eyes on the change,
> +without demanding that every reviewer have the same amount of
> +expertise as the area maintainer. This way one can review for general
> +sanity, accurate comments, obvious mistakes, etc, without being forced
> +to assert "Yes, I understand these changes in every detail and have
> +tested them."

Just to be completely above-board here:

My original proposal, which seems to have been accepted pretty well,
was only about loosening our system for the bindings changes.

As I was writing, though, I realized all the same reasoning applied to
the tools area. Thinking it unlikely anyone would disagree, I just
included that in the first pass above. Hope that's okay with all.

-K

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Thu Jan 15 18:17:12 2004

This is an archived mail posted to the Subversion Dev mailing list.