Attention is currently required from: Alexander Goncharov, Angel Pons, Edward O'Callaghan, Peter Marheine, Stefan Reinauer, Swift Geek (Sebastian Grzywna), Thomas Heijligen.
Anastasia Klimchuk has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/75906?usp=email )
Change subject: doc: Add new Development guidelines ......................................................................
Patch Set 2:
(3 comments)
Patchset:
PS1:
As a reference we have done a brainstorming session https://docs.google. […]
Also I want to mention that some sections that used to be a part of Dev guidelines will be moved into other pages: How to add a chip Release process History
One thing that potentially could be added, is one paragraph about branches, just general info. Not in the beginning, but at the end of guidelines. What do people think, do we need some info about branches in this doc?
File doc/dev_guide/development_guidelines.rst:
https://review.coreboot.org/c/flashrom/+/75906/comment/b695bd67_79eab45d : PS1, Line 2: Development Guidelilnes
typo
Done, thanks for catching
https://review.coreboot.org/c/flashrom/+/75906/comment/04878dcc_a11e38ea : PS1, Line 5: Intro, is there anything useful to say?
Is that a leftover from some draft?
Yes and no: I was thinking whether the intro is needed, and and if yes what should be there. I didn't have a clear idea what to say in the intro, so decided to push the patch like this in case other people have ideas. If no one thinks it's needed, I will just delete this line.
-
Anastasia Klimchuk (Code Review)