Attention is currently required from: Alexander Goncharov, Anastasia Klimchuk, Edward O'Callaghan, Peter Marheine, Stefan Reinauer, Swift Geek (Sebastian Grzywna), Thomas Heijligen.
3 comments:
File doc/dev_guide/development_guidelines.rst:
Patch Set #1, Line 5: Intro, is there anything useful to say?
No idea what to put in an introduction, maybe:
This document describes the rules and recommendations about the development, contribution and review processes.
Note: it may be useful to visibly mark open questions in a patch with a `TODO` prefix or similar, so as to (hopefully) catch the reviewers' attention.
File doc/dev_guide/development_guidelines.rst:
Patch Set #2, Line 7: Set up the git repository and dev environment
+1, having detailed instructions in a separate "how-to" document (or multiple) would make more sense. To us, a guideline would be something containing general rules, principles and/or pieces of advice about something (in this case, the development/contribution/review processes).
One could link to the how-to's with something like this:
For instructions on how to clone the flashrom repository for development, please refer to the <link to "how-to clone" page>.
Patch Set #2, Line 279: Except, if a commit
The comma seems rather clunky, how about:
Exception: if a commit is ...
To view, visit change 75906. To unsubscribe, or for help writing mail filters, visit settings.