From: Ævar Arnfjörð Bjarmason Date: Tue, 20 Apr 2010 18:36:51 +0000 (+0000) Subject: Commit messages should start with a ~50 char summary X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=commitdiff_plain;h=1a5f98ea6257ef63e515f48e05311fba2cdd7dbc;p=p5sagit%2Fp5-mst-13.2.git Commit messages should start with a ~50 char summary The first sentence is stolen from Git's own Documentation/SubmittingPatches. We have a lot of commit messages that overflow `git log --pretty=oneline', but it wouldn't hurt if we change that sooner rather than later. --- diff --git a/pod/perlrepository.pod b/pod/perlrepository.pod index 516e762..f23d6a6 100644 --- a/pod/perlrepository.pod +++ b/pod/perlrepository.pod @@ -439,8 +439,16 @@ you need to do. As you craft each patch you intend to submit to the Perl core, it's important to write a good commit message. -Your commit message should start with a description of the problem that -the patch corrects or new functionality that the patch adds. +The first line of the commit message should be a short description and +should skip the full stop. It should be no longer than the subject +line of an E-Mail, 50 characters being a good rule of thumb. + +A lot of Git tools (Gitweb, GitHub, git log --pretty=oneline, ..) will +only display the first line (cut off at 50 characters) when presenting +commit summaries. + +The commit message should include description of the problem that the +patch corrects or new functionality that the patch adds. As a general rule of thumb, your commit message should let a programmer with a reasonable familiarity with the Perl core quickly understand what