Differences between revisions 12 and 14 (spanning 2 versions)
Revision 12 as of 2012-10-23 06:53:01
Size: 1584
Comment: link to ContributingChanges#Patch_descriptions
Revision 14 as of 2013-01-21 23:46:21
Size: 1672
Editor: rcl
Comment: updated style
Deletions are marked like this. Additions are marked like this.
Line 1: Line 1:
comments for contributing to mercurial see [[ContributingChanges#Patch_descriptions|Patch descriptions]]

== Writing good changeset comments ==
= Writing Good Changeset Comments =
<<TableOfContents>>
Line 13: Line 12:
== Good Example ==
Line 29: Line 29:
== Bad Example ==
Line 44: Line 45:

== See also ==
 * For comments on contributed patches to Mercurial, see [[ContributingChanges#Patch_descriptions|Patch descriptions]]

Writing Good Changeset Comments

Many Mercurial tools use the first line of a changeset comment as a short-form description of the changeset.

You should thus treat the first line like the subject line of an email message. It should:

  • stand alone,
  • be less than 65 characters long, and
  • give a quick idea of the content of the changeset.

1. Good Example

Here is an example of a good changeset comment:

 First stab at the veeblefrotzer subsystem

 Implemented using the McWhirly/O'Blivet technique.

 Displays correctly on the twelfth of every month,
 but doesn't yet work during the other 27-30 days.

This has a few things going for it:

  • The first line succinctly describes the change.
  • The entire comment is short and informative.
  • It mentions known shortcomings.

2. Bad Example

Here's a bad example:

 Implemented the veeblefrotzer subsystem, which uses the
 McWhirly/O'Blivet scheme for envolvolution of the subducted
 whingnangle.  See McWhirly, O'Blivet, "Acta Exsanguinata", vol. III,
 chap. 19 for a concise description of the technique, unless you can't
 find the chapter because someone has razored it out and papered it to
 the walls of their cube.

What's so bad about it?

  • First line doesn't stand alone
  • No paragraph breaks → poor readability

  • Loads of uninformative detail

3. See also


CategoryHowTo

ภาษาไทย, 日本語

ChangeSetComments (last edited 2013-01-24 04:46:17 by rcl)