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

Re: [PATCH] 1.5.x branch: python/cmdline

From: Karl Fogel <kfogel_at_red-bean.com>
Date: Mon, 02 Jun 2008 21:56:02 -0400

Lieven Govaerts <svnlgo_at_mobsol.be> writes:
> As Daniel said, our log messages are normally a bit shorter ;) Just a
> few of the key phrases should be sufficient.

Also, remember that the key thing is not length -- it's just that most
information should go in the code (as comments) instead of in the log
message.

The log message should be an overview of the change: it prepares the
reviewer's mind for reading the code change, by explaining what
motivated the change and then briefly describing the overall shape of
the change. But detailed information should be in the change itself.

So Alexander, when you're writing future log messages, ask yourself as
you write "Does any of this information belong in the code instead of in
the log message?"

(I did read the rest of this thread, by the way.)

-Karl

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-06-03 03:56:52 CEST

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

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.