Commit Digest/Developer guidelines: Difference between revisions
(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 |
||
| (4 intermediate revisions by the same user not shown) | |||
| Line 1: | Line 1: | ||
[[Commit Digest| << back]] | |||
Some notes for developers on how to increase chances that a commit message is included in the Digest: | 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 | * add a description text why a commit is interesting (see examples below), | ||
* 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. | |||
'''(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 18: | ||
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''' | |||
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. | ||
Latest revision as of 10:07, 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),
- 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.