diff options
author | Joseph Herlant <aerostitch@users.noreply.github.com> | 2019-08-23 08:00:20 -0700 |
---|---|---|
committer | GitHub <noreply@github.com> | 2019-08-23 08:00:20 -0700 |
commit | 305d5f82bd4024f4325034e828b29ace0f27373f (patch) | |
tree | bc81fccaf21cb5ff62a846063a5116ad418c1376 /docs | |
parent | 5e644dad94fbbe287aece4bf04428265349eafe4 (diff) | |
download | navit-305d5f82bd4024f4325034e828b29ace0f27373f.tar.gz |
Add:doc:Migrate the commit guidelines document to readthedocs (#841)
Diffstat (limited to 'docs')
-rw-r--r-- | docs/development/commit_guidelines.rst | 54 | ||||
-rw-r--r-- | docs/index.rst | 1 |
2 files changed, 55 insertions, 0 deletions
diff --git a/docs/development/commit_guidelines.rst b/docs/development/commit_guidelines.rst new file mode 100644 index 000000000..dff76fac2 --- /dev/null +++ b/docs/development/commit_guidelines.rst @@ -0,0 +1,54 @@ +====================== +Commit guidelines +====================== + +Here are the current guidelines if you want to commit something. Please also read the [programming guidelines](../programming_guidelines.html) + +'Core' components changes +========================= + +Do not modify a 'core' component without discussing it first with the project leads. + +Core components include data structures, configuration handling. If you are unsure, just ask. + +Commit per feature +================== + +When committing, try to have one commit per feature (or per meaningful part of a larger feature). The goal is to always have working code; at least make sure each commit leaves the repository in a compilable state. + +Also avoid putting multiple, independent changes into one commit. +Thus if you have multiple, independent changes in your local working copy, avoid committing a whole folder at once, especially Navit's Sourcecode root. Instead, explicitly select the files for each commit. + +Format of the commit log +======================== + +Since we are too lazy to maintain a Changelog, we have a script which parses the commit logs and generate a Changelog for us. + +We have agreed about using the following syntax : ` <Action>:<component>:<log message>[|Optional comments]` + +Examples : + Fix:Core:Fixed nasty bug in ticket #134 + Fix:GTK:Fixed nasty bug about destination button|Thanks someguy for the patch! + +Action can be something like: +* Fix (bug fix) +* Add (new feature) +* Patch +* Refactoring (does not change behavior of the program) + +It allows the changes to be sorted by categories + +The most common components are: +* core +* gui/gtk +* gui/internal +* graphics/gtk +* graphics/qt_qpainter +* graphics/opengl +* mapdriver +* tools + +The comment part is optional. Useful for when applying a patch for example, and giving credits. +The part after `|` will not appear in the wiki. + +About the log message, it's up to you :) diff --git a/docs/index.rst b/docs/index.rst index ac1a5ab66..5c239bd4b 100644 --- a/docs/index.rst +++ b/docs/index.rst @@ -46,3 +46,4 @@ Navit is highly customizable, from map layouts and on-screen display to the deta :caption: Navit's developer documentation development/programming_guidelines + development/commit_guidelines |