[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Fix error messages in website build (issue4428077)
From: |
percival . music . ca |
Subject: |
Re: Fix error messages in website build (issue4428077) |
Date: |
Sat, 07 May 2011 01:13:54 +0000 |
also, could we change the subject of the commit? (either by clever git
stuff, or by starting a new commit on a separate git branch --
personally, I'd do the latter)
Something like:
Suppress known "missing" warnings in extract_texi
(remember about the 50-char soft limit)
The point behind the past two comments is that when you're looking at
build system changes (or any changes, really), it's nice if you can see
a nice short message that's really descriptive. When people look at the
history, they're generally looking at dozens of commits. Having good
subject messages (and a good amount of material in each commit) can make
the changes a lot easier to understand.
http://codereview.appspot.com/4428077/
- Re: Fix error messages in website build (issue4428077), (continued)
- Re: Fix error messages in website build (issue4428077), percival . music . ca, 2011/05/02
- Re: Fix error messages in website build (issue4428077), PhilEHolmes, 2011/05/05
- Re: Fix error messages in website build (issue4428077), percival . music . ca, 2011/05/05
- Re: Fix error messages in website build (issue4428077), PhilEHolmes, 2011/05/06
- Re: Fix error messages in website build (issue4428077), percival . music . ca, 2011/05/06
- Re: Fix error messages in website build (issue4428077),
percival . music . ca <=
- Re: Fix error messages in website build (issue4428077), tdanielsmusic, 2011/05/07