[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Gets rid of length in the docs. (issue 4965053)
From: |
Reinhold Kainhofer |
Subject: |
Re: Gets rid of length in the docs. (issue 4965053) |
Date: |
Mon, 29 Aug 2011 12:55:07 +0200 |
User-agent: |
KMail/1.13.6 (Linux/2.6.38-11-generic; KDE/4.7.0; i686; ; ) |
Am Montag, 29. August 2011, 11:56:17 schrieb Janek Warchoł:
> From my point of view, commit messages, descriptions in tracker and on
> Rietveld are essential. Some patches have no description and i cannot
> say anything about them.
+1. Also, Rietveld descriptions like "Fix 1234" are NOT useful at all. It
requires you to look up what bug 1234 is about, and then try to understand how
the implementation tried to fix it.
The same goes for git commit messages. It happens regularly that I have to
look up who last changed something, and then commit message does not tell
anything about WHY a change was done at all.
Cheers,
Reinhold
--
------------------------------------------------------------------
Reinhold Kainhofer, address@hidden, http://reinhold.kainhofer.com/
* Financial & Actuarial Math., Vienna Univ. of Technology, Austria
* http://www.fam.tuwien.ac.at/, DVR: 0005886
* LilyPond, Music typesetting, http://www.lilypond.org
- Re: Gets rid of length in the docs. (issue 4965053), (continued)
- Re: Gets rid of length in the docs. (issue 4965053), Trevor Daniels, 2011/08/30
- Re: Gets rid of length in the docs. (issue 4965053), Keith OHara, 2011/08/30
- Re: Gets rid of length in the docs. (issue 4965053), Mike Solomon, 2011/08/30
- Re: Gets rid of length in the docs. (issue 4965053), David Kastrup, 2011/08/30
- Re: Gets rid of length in the docs. (issue 4965053), Trevor Daniels, 2011/08/30
- Re: Gets rid of length in the docs. (issue 4965053), Reinhold Kainhofer, 2011/08/30
- Re: Gets rid of length in the docs. (issue 4965053), Trevor Daniels, 2011/08/29
- Re: Gets rid of length in the docs. (issue 4965053), Graham Percival, 2011/08/29
- Re: Gets rid of length in the docs. (issue 4965053), Mike Solomon, 2011/08/29
- Re: Gets rid of length in the docs. (issue 4965053), Janek Warchoł, 2011/08/29
- Re: Gets rid of length in the docs. (issue 4965053),
Reinhold Kainhofer <=
- Re: Gets rid of length in the docs. (issue 4965053), Trevor Daniels, 2011/08/29
- summarizing patches for review (was: Gets rid of length in the docs. (issue 4965053)), Graham Percival, 2011/08/29
- Re: summarizing patches for review (was: Gets rid of length in the docs. (issue 4965053)), Trevor Daniels, 2011/08/30
- Re: summarizing patches for review (was: Gets rid of length in the docs. (issue 4965053)), Graham Percival, 2011/08/31
- Re: summarizing patches for review (was: Gets rid of length in the docs. (issue 4965053)), Trevor Daniels, 2011/08/31