2019 01 21 Open NEST Developer Video Conference - nest/nest-simulator GitHub Wiki
Agenda
-
Welcome
-
Info/Discussion on git commit messages (speaker @clinssen)
-
some commit messages only make sense in the context of current development in the specific branch, but is mostly useless after merge
-
blame based on commits
-
distinguish between what changed (diff) and why it changed (← commit messages!!)
-
has to be part of code review
-
do not bundle many different changes in the same commit
-
6 1/2 rules for messages
- correct name and mail address
- atomic commits: separate commits for logically separate changes
- "if applied, this commit will…" {commit message}
- be descriptive: why (explain), how (justify), what (effects)
- be concise (single line <50 <70 chars (git limit))
- cross-reference where possible
- have consistent style, capital letter?, full stop?, …
-
-
Review of NEST User Mailing List
- little activity
-
Review of open Github Pull Request
- PRs checked
- updates on projects
-
Review of open Github Issues