Patrick Georgi has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31127 )
Change subject: Documentation: describe coreboot on the dev site's landing page ......................................................................
Patch Set 7:
(8 comments)
Many thanks for the reviews!
https://review.coreboot.org/#/c/31127/6/Documentation/index.md File Documentation/index.md:
https://review.coreboot.org/#/c/31127/6/Documentation/index.md@23 PS6, Line 23: payloads
Reading the commit message, it still needs to be written in follow-up commits.
That's right, but I added something now (and removed the blurb from the commit message).
https://review.coreboot.org/#/c/31127/6/Documentation/index.md@34 PS6, Line 34: eg
e.g.
Done
https://review.coreboot.org/#/c/31127/6/Documentation/index.md@37 PS6, Line 37: eg.
e.g.
Done
https://review.coreboot.org/#/c/31127/6/Documentation/index.md@88 PS6, Line 88: eg.
e.g.
Done
https://review.coreboot.org/#/c/31127/6/Documentation/index.md@89 PS6, Line 89: eg.
e.g.
Done
https://review.coreboot.org/#/c/31127/6/Documentation/index.md@125 PS6, Line 125: DSCM
I’d spell it out.
I went for "version control system" - too many names for that kind of software :-)
https://review.coreboot.org/#/c/31127/6/Documentation/index.md@134 PS6, Line 134:
Maybe add a comma
Done
https://review.coreboot.org/#/c/31127/6/Documentation/index.md@135 PS6, Line 135: eg
e.g.
Done