Differences between revisions 12 and 13
Revision 12 as of 2012-10-23 06:53:01
Size: 1584
Comment: link to ContributingChanges#Patch_descriptions
Revision 13 as of 2012-10-23 16:06:37
Size: 1608
Editor: mpm
Comment:
Deletions are marked like this. Additions are marked like this.
Line 1: Line 1:
<<Include(A:style)>>

{i} This page does not meet our wiki style guidelines. Please help improve this page by cleaning up its formatting.

comments for contributing to mercurial see 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.

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.

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


CategoryHowTo

ภาษาไทย, 日本語

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