Message197966
| Author |
georg.brandl |
| Recipients |
brett.cannon, ezio.melotti, georg.brandl, lemburg, ncoghlan, orsenthil, pitrou, r.david.murray, tshepang |
| Date |
2013年09月17日.07:55:54 |
| SpamBayes Score |
-1.0 |
| Marked as misclassified |
Yes |
| Message-id |
<1379404555.3.0.92987928561.issue18967@psf.upfronthosting.co.za> |
| In-reply-to |
| Content |
[MAL]
> * Commit messages only provide a very terse hint at what a
> particular patch set was meant for. The target audience
> is other core developers.
>
> * News entries explain these patches (there may be more than one
> for a particular issue or project) to Python programmers who
> need to know what to watch out for when upgrading to a new
> version.
I agree. But if you have to write two different versions, putting them
both in the commit message has two advantages:
- it's arguably easier not to have to touch another file
- the extended explanation can be useful for core developers too :)
In any case, I would very much like to see a unified style for commit messages a la Mercurial: a very terse, but self-sufficient first line, and more explanation below.
Not to speak of the non-arguable advantage of getting rid of merge conflicts in Misc/NEWS.
So if the script had a mode to "set off" the intended NEWS entry from both a header (the terse 1-line commit message) and an optional footer (explanation of technical details only of interest to committers), I would say +1. |
|