From: Ted Zlatanov <tzz@lifelogs.com>
To: ding@gnus.org
Subject: Re: Git commit methodology
Date: Mon, 30 Aug 2010 08:48:13 -0500 [thread overview]
Message-ID: <87pqx0p5ya.fsf@lifelogs.com> (raw)
In-Reply-To: <m362yscjae.fsf@quimbies.gnus.org>
On Mon, 30 Aug 2010 15:38:33 +0200 Lars Magne Ingebrigtsen <larsi@gnus.org> wrote:
LMI> Ted Zlatanov <tzz@lifelogs.com> writes:
>> The ChangeLog format is the same. The *commit message* should be
>>
>> --cut--
>> short-description-of-change
>>
>> normal-commit-message
>> --cut--
>>
>> which produces nice one-line summaries in the condensed git log (and
>> Bazaar).
LMI> Ah, right. Has anybody done any ChangeLog/vc log integration? I mean,
LMI> if I'm checking in a change set that includes the ChangeLog, then it
LMI> would make sense to default the text to text extracted from the
LMI> ChangeLog.
LMI> That is, if the ChangeLog text about to be checked in is:
LMI> -------
LMI> 2010-08-29 Lars Magne Ingebrigtsen <larsi@gnus.org>
LMI> * gnus-start.el (gnus-dribble-read-file): Ensure that the directory
LMI> where the dribbel file lives exists.
LMI> -------
LMI> then the vc log buffer could be populated with:
LMI> -------
LMI> Ensure that the directory where the dribble file lives exists.
LMI> -------
This works for a one-sentence commit, but generally it doesn't. The
summary should be very succinct and ChangeLog messages tend to be
explicit and long. I think it's better to type the summary manually,
unless the one-sentence match works. I don't think anyone has done work
on this but it does sound useful.
LMI> (Note how the code also fixed up the typo automatically. :-)
I'd rather see it introduce typos automatically. Then no one will ever
know you screwed up :)
Ted
next prev parent reply other threads:[~2010-08-30 13:48 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-08-29 14:41 Lars Magne Ingebrigtsen
2010-08-30 13:31 ` Ted Zlatanov
2010-08-30 13:38 ` Lars Magne Ingebrigtsen
2010-08-30 13:48 ` Ted Zlatanov [this message]
2010-08-30 16:16 ` Lars Magne Ingebrigtsen
2010-08-30 17:03 ` Lars Magne Ingebrigtsen
2010-08-30 17:10 ` Leo
2010-08-30 17:12 ` Lars Magne Ingebrigtsen
2010-08-30 17:18 ` Leo
2010-08-30 17:22 ` Lars Magne Ingebrigtsen
2010-08-30 17:34 ` Leo
2010-08-30 17:51 ` Lars Magne Ingebrigtsen
2010-08-30 20:16 ` Lars Magne Ingebrigtsen
2010-08-30 18:51 ` Steinar Bang
2010-08-31 18:47 ` Ted Zlatanov
2010-08-31 18:51 ` Lars Magne Ingebrigtsen
2010-09-01 13:50 ` Ted Zlatanov
2010-09-01 14:42 ` Lars Magne Ingebrigtsen
2010-09-01 15:06 ` Ted Zlatanov
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87pqx0p5ya.fsf@lifelogs.com \
--to=tzz@lifelogs.com \
--cc=ding@gnus.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).