Commit Digest/Developer guidelines: Difference between revisions

From KDE Community Wiki
(Created page with 'Some notes for developers on how to increase chances that a commit message is included in the Digest: * add a description text why a commit is interesting (see examples below of...')
 
No edit summary
Line 2: Line 2:


* add a description text why a commit is interesting (see examples below of good descriptions),
* add a description text why a commit is interesting (see examples below of good descriptions),
* most of the description text should be understandable for non-programmers
* refer to a bug report / wishlist item (if applicable),
* refer to a bug report / wishlist item (if applicable),
* reference a screenshot / blog posting,
* reference a screenshot / blog posting,
* include the "DIGEST" keyword (note: all capital letters) in your commit message.
* include the "DIGEST" keyword (note: all capital letters) in your commit message.


== Examples of good commit message ==
== Examples of good commit message ==
The examples (1), (2) contain a one-liner summary and an explanation about the problem that is solved with the commit. In both cases the text is understandable for non-programmers.


=== Re-add KFind shortcut for the Dolphin KPart ===
'''(1)  Re-add KFind shortcut for the Dolphin KPart'''


When the searching has been integrated into Dolphin the "Find File..."
When the searching has been integrated into Dolphin the "Find File..."
Line 17: Line 16:
KPart so that it is still available for Konqueror.
KPart so that it is still available for Konqueror.


 
'''(2)  Let each DolphinMainWindow run in a custom process'''
=== Let each DolphinMainWindow run in a custom process ===


A possible crash in one Dolphin window should not result in crashing  
A possible crash in one Dolphin window should not result in crashing  
other Dolphin windows. Beside this it also prevents issues with  
other Dolphin windows. Beside this it also prevents issues with  
modal dialogs or notifications.
modal dialogs or notifications.

Revision as of 10:03, 25 April 2011

Some notes for developers on how to increase chances that a commit message is included in the Digest:

  • add a description text why a commit is interesting (see examples below of good descriptions),
  • most of the description text should be understandable for non-programmers
  • refer to a bug report / wishlist item (if applicable),
  • reference a screenshot / blog posting,
  • include the "DIGEST" keyword (note: all capital letters) in your commit message.

Examples of good commit message

The examples (1), (2) contain a one-liner summary and an explanation about the problem that is solved with the commit. In both cases the text is understandable for non-programmers.

(1) Re-add KFind shortcut for the Dolphin KPart

When the searching has been integrated into Dolphin the "Find File..." entry had been removed from the Tools menu. Re-add it into the Dolphin KPart so that it is still available for Konqueror.

(2) Let each DolphinMainWindow run in a custom process

A possible crash in one Dolphin window should not result in crashing other Dolphin windows. Beside this it also prevents issues with modal dialogs or notifications.