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

Re: Moving some of our tools to "main" subversion

From: Ivan Zhakov <ivan_at_visualsvn.com>
Date: Mon, 1 Sep 2014 15:20:34 +0400

On 29 August 2014 22:48, Branko Čibej <brane_at_wandisco.com> wrote:
> On 29.08.2014 20:12, Julian Foad wrote:
> Please everyone calm down.
> From scanning this conversation, it looks like the main technical substance
> is that a commit exposed some symbols wrongly, and the layering of the
> functionality may need to be improved. Or something like that. (Exactly what
> the technical issues are, is not the point of this email.)
> ** This is not a crisis. **
> I'm referring to the fact that we're in the situation that
> one developer's commits are a priori considered suspect by
> another developer.

I do not understand this point. I'm reading everyone's commits and
this is not my fault if I find too much mistakes in the code produced
by the particular developer.

Moreover, I never a priory treat any commits as a good ones (no matter
who the author is). But please do not think that I'm trying to say
that I'm the smartest developer here. I'm sure that the reviewer
should treat any code modification as a potentially wrong one. The
code review is clearly useless otherwise.

From my point of view, the system works because each developer should
be the first prejudiced reviewer for all his new code. And I hope you
will not suggest me to filter out someone's commits from my reading
list in order to have such problems solved.

Ivan Zhakov
Received on 2014-09-01 13:21:24 CEST

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