Warning: preg_match(): Compilation failed: group name must start with a non-digit at offset 8 in /home/project-web/netatalk/htdocs/wiki/includes/MagicWord.php on line 739

Warning: preg_match(): Compilation failed: group name must start with a non-digit at offset 8 in /home/project-web/netatalk/htdocs/wiki/includes/MagicWord.php on line 739

Warning: preg_match_all(): Compilation failed: group name must start with a non-digit at offset 4 in /home/project-web/netatalk/htdocs/wiki/includes/MagicWord.php on line 717

Warning: Invalid argument supplied for foreach() in /home/project-web/netatalk/htdocs/wiki/includes/MagicWord.php on line 718

Warning: preg_replace(): Compilation failed: group name must start with a non-digit at offset 4 in /home/project-web/netatalk/htdocs/wiki/includes/MagicWord.php on line 722

Warning: preg_match_all(): Compilation failed: group name must start with a non-digit at offset 4 in /home/project-web/netatalk/htdocs/wiki/includes/MagicWord.php on line 717

Warning: Invalid argument supplied for foreach() in /home/project-web/netatalk/htdocs/wiki/includes/MagicWord.php on line 718

Warning: preg_replace(): Compilation failed: group name must start with a non-digit at offset 4 in /home/project-web/netatalk/htdocs/wiki/includes/MagicWord.php on line 722

Warning: preg_match(): Compilation failed: group name must start with a non-digit at offset 8 in /home/project-web/netatalk/htdocs/wiki/includes/MagicWord.php on line 739

Warning: preg_match(): Compilation failed: group name must start with a non-digit at offset 8 in /home/project-web/netatalk/htdocs/wiki/includes/MagicWord.php on line 739

Warning: preg_match(): Compilation failed: group name must start with a non-digit at offset 8 in /home/project-web/netatalk/htdocs/wiki/includes/MagicWord.php on line 739

Warning: preg_match_all(): Compilation failed: group name must start with a non-digit at offset 4 in /home/project-web/netatalk/htdocs/wiki/includes/MagicWord.php on line 717

Warning: Invalid argument supplied for foreach() in /home/project-web/netatalk/htdocs/wiki/includes/MagicWord.php on line 718

Warning: preg_replace(): Compilation failed: group name must start with a non-digit at offset 4 in /home/project-web/netatalk/htdocs/wiki/includes/MagicWord.php on line 722

Warning: preg_match_all(): Compilation failed: group name must start with a non-digit at offset 4 in /home/project-web/netatalk/htdocs/wiki/includes/MagicWord.php on line 717

Warning: Invalid argument supplied for foreach() in /home/project-web/netatalk/htdocs/wiki/includes/MagicWord.php on line 718

Warning: preg_replace(): Compilation failed: group name must start with a non-digit at offset 4 in /home/project-web/netatalk/htdocs/wiki/includes/MagicWord.php on line 722

Warning: preg_match(): Compilation failed: group name must start with a non-digit at offset 8 in /home/project-web/netatalk/htdocs/wiki/includes/MagicWord.php on line 739

Warning: preg_match(): Compilation failed: group name must start with a non-digit at offset 8 in /home/project-web/netatalk/htdocs/wiki/includes/MagicWord.php on line 739
Developer Infos - Netatalk Wiki

Developer Infos


Warning: preg_match(): Compilation failed: group name must start with a non-digit at offset 8 in /home/project-web/netatalk/htdocs/wiki/includes/MagicWord.php on line 739
From Netatalk Wiki
Jump to: navigation, search
(Main Netatalk Branches)
 
Line 32: Line 32:
 
* the bugreport must be referenced in the commit message
 
* the bugreport must be referenced in the commit message
 
* no branch merges
 
* no branch merges
 +
 +
== Review process ==
 +
 +
We now require formal review of all patches.
 +
 +
The author of patch should add a signed-off tag and the reviewer adds a reviewed-by tag. Very formal, but it encourages better coding and documentation.
 +
 +
This means every commit in master should have been reviewed by two team members (if the author is a team member, only one review by another team member needed).
 +
 +
This is the same process used in [https://wiki.samba.org/index.php/CodeReview Samba]
 +
 +
== Commit messages ==
 +
 +
Commit messages should have a short, descriptive first summary line that begins with the affected component, eg
 +
 +
  afpd: new options "force user" and "force group"
 +
 +
This is helpful when browing a git log in oneline mode.
 +
 +
Then the commit message should explain what the change is about, the more, the better.
 +
 +
At the end the author adds his signed-off tag.
  
 
== Basic Netatalk Git ==
 
== Basic Netatalk Git ==

Latest revision as of 13:42, 18 June 2016

Personal tools
Namespaces
Variants
Actions
Navigation
Toolbox