Matt DeVillier has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31252 )
Change subject: Documentation: update/improve distribution listing ......................................................................
Patch Set 1:
(9 comments)
https://review.coreboot.org/#/c/31252/1/Documentation/distributions.md File Documentation/distributions.md:
https://review.coreboot.org/#/c/31252/1/Documentation/distributions.md@15 PS1, Line 15: blob-free OS and coreboot firmware
Huh? Don't they use the FSP?
blob-free *OS*
https://review.coreboot.org/#/c/31252/1/Documentation/distributions.md@21 PS1, Line 21: on
onwards, maybe?
Done
https://review.coreboot.org/#/c/31252/1/Documentation/distributions.md@22 PS1, Line 22:
running *on* the
Done
https://review.coreboot.org/#/c/31252/1/Documentation/distributions.md@28 PS1, Line 28: [Libretrend](https://libretrend.com) sells the Librebox, a NUC-like PC
This line is noticeably shorter than the rest
Done
https://review.coreboot.org/#/c/31252/1/Documentation/distributions.md@29 PS1, Line 29: chips
ships
Done
https://review.coreboot.org/#/c/31252/1/Documentation/distributions.md@37 PS1, Line 37: /
This slash looks terrible in here
seems like a style critique ;-)
https://review.coreboot.org/#/c/31252/1/Documentation/distributions.md@43 PS1, Line 43: Mr Chromebox
Maybe add the period after "Mr"
while grammatically correct, is not how my nick appears anywhere online. I'll use camel-case for consistency
https://review.coreboot.org/#/c/31252/1/Documentation/distributions.md@48 PS1, Line 48: ,,
double comma? should be dot and comma (.,) to make "i.e. […]
Done
https://review.coreboot.org/#/c/31252/1/Documentation/distributions.md@59 PS1, Line 59: a good deal
maybe use "significantly" instead
Done