Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [gef-dev] Git Commit Logs

Hi Ed, 

yes, that's a point. However, I simply wanted to emphasize that it should not match be the bug title but be a reasonable commit message. Whether the WHAT or the WHY is the more important information may probably differ from case to case so I think the committer should have the freedom to decide.

Cheers
Alexander

Am 25.02.2012 um 15:57 schrieb Ed Willink:

Hi Alexander

Somewhere, I think in the EGIT guide, is the recommendation that the commit message should be

WHY the change was made

since WHAT was changed can be reconstructed, but the motivation is easily lost forever.

    Regards

        Ed Willink


On 25/02/2012 13:49, Alexander Nyssen wrote:
Hi all,

I just wanted to start reworking the Contributor Guide to reflect our changed project infrastructure. 

As it provides guidance on how to format commit logs, I think we should first agree on how we want to do this for GEF (and GEF4) in the future. I think the old cvs-inspired format of kind

 [213359] gef-head crevells 071221 Make GEF's snapping and tools extensible to allow moving shapes with arrow keys

seems to be a bit cumbersome, now that we have Git in place. I would like to propose something like the following:

[<bug-id>] (<contributor>)? <commit-message>.

where <bug-id> is the id of the bug the commit is related to, or NONE in case the commit is not directly related to any bug. <contributor> is the full name of the contributor in case the contribution is not done by the committer itself. In case the committer also is the contributor it may be left out. The commit message should not equal the title of the related bug (one can read it there) but should instead describe the performed changes. Two  examples:

[355997] (Matthias Wienand) Added support for calculating bezier curve intersections by means of bezier clipping algorithm. (CQ 5976)
[NONE] Updated target definition to Eclipse SDK 4.2.0.I20120222-0915.

What do you think?
--
Dr. Alexander Nyßen
Dipl.-Inform.
Software-Engineer

Telefon: +49 (0) 231 / 98 60-210
Telefax: +49 (0) 231 / 98 60-211
Mobil: +49 (0) 151 /  17396743

http://www.itemis.de 
alexander.nyssen@xxxxxxxxx 

itemis AG
Am Brambusch 15-24
44536 Lünen

Rechtlicher Hinweis:

Amtsgericht Dortmund, HRB 20621

Vorstand: Jens Wagener (Vors.), Wolfgang Neuhaus, Dr. Georg Pietrek, Jens Trompeter, Sebastian Neus

Aufsichtsrat: Dr. Burkhard Igel (Vors.), Stephan Grollmann, Michael Neuhaus




_______________________________________________
gef-dev mailing list
gef-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/gef-dev


-----
No virus found in this message.
Checked by AVG - www.avg.com
Version: 2012.0.1913 / Virus Database: 2114/4830 - Release Date: 02/24/12

_______________________________________________
gef-dev mailing list
gef-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/gef-dev

--  
Dr. Alexander Nyßen
Dipl.-Inform.
Software-Engineer

Telefon: +49 (0) 231 / 98 60-210
Telefax: +49 (0) 231 / 98 60-211
Mobil: +49 (0) 151 /  17396743


itemis AG
Am Brambusch 15-24
44536 Lünen

Rechtlicher Hinweis:

Amtsgericht Dortmund, HRB 20621

Vorstand: Jens Wagener (Vors.), Wolfgang Neuhaus, Dr. Georg Pietrek, Jens Trompeter, Sebastian Neus

Aufsichtsrat: Dr. Burkhard Igel (Vors.), Stephan Grollmann, Michael Neuhaus


Back to the top