Felix Singer has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/40662 )
Change subject: documentation: Add documentation ideas for season of docs ......................................................................
Patch Set 8: Code-Review+1
(22 comments)
https://review.coreboot.org/c/coreboot/+/40662/8/Documentation/contributing/... File Documentation/contributing/documentation_ideas.md:
https://review.coreboot.org/c/coreboot/+/40662/8/Documentation/contributing/... PS8, Line 1: # Documentation Ideas I would avoid camel case for the headlines.
https://review.coreboot.org/c/coreboot/+/40662/8/Documentation/contributing/... PS8, Line 3: to improve for improving
https://review.coreboot.org/c/coreboot/+/40662/8/Documentation/contributing/... PS8, Line 11: entry proposal?
https://review.coreboot.org/c/coreboot/+/40662/8/Documentation/contributing/... PS8, Line 12: They : should also list people interested in supporting people who want to work : on them. I would rephrase that. Maybe like this?
Also, a project should have at least one person acting as a mentor for supporting the person who works on it.
https://review.coreboot.org/c/coreboot/+/40662/8/Documentation/contributing/... PS8, Line 16: Restructure Restructuring
https://review.coreboot.org/c/coreboot/+/40662/8/Documentation/contributing/... PS8, Line 18: structure I would write a "to" before that. Sounds better IMO.
https://review.coreboot.org/c/coreboot/+/40662/8/Documentation/contributing/... PS8, Line 22: to restructure the documentation is to split it up such : that we divide the group of users into: for the restructuring is to split the documentation up into sections targeting different groups of people and users.
https://review.coreboot.org/c/coreboot/+/40662/8/Documentation/contributing/... PS8, Line 27: coreboot it
https://review.coreboot.org/c/coreboot/+/40662/8/Documentation/contributing/... PS8, Line 32: of : coreboot I would remove that, since it was mentioned right before.
https://review.coreboot.org/c/coreboot/+/40662/8/Documentation/contributing/... PS8, Line 37: the developers section it
https://review.coreboot.org/c/coreboot/+/40662/8/Documentation/contributing/... PS8, Line 59: s singular
https://review.coreboot.org/c/coreboot/+/40662/8/Documentation/contributing/... PS8, Line 61: setting coreboot up setting up coreboot
https://review.coreboot.org/c/coreboot/+/40662/8/Documentation/contributing/... PS8, Line 62: E e
https://review.coreboot.org/c/coreboot/+/40662/8/Documentation/contributing/... PS8, Line 64: G g
https://review.coreboot.org/c/coreboot/+/40662/8/Documentation/contributing/... PS8, Line 64: needs an update/extensions needs improvements on the usability
https://review.coreboot.org/c/coreboot/+/40662/8/Documentation/contributing/... PS8, Line 67: technique technique_s_
https://review.coreboot.org/c/coreboot/+/40662/8/Documentation/contributing/... PS8, Line 84: support adding support
https://review.coreboot.org/c/coreboot/+/40662/8/Documentation/contributing/... PS8, Line 107: Tinanocore TianoCore
https://review.coreboot.org/c/coreboot/+/40662/8/Documentation/contributing/... PS8, Line 107: b B
https://review.coreboot.org/c/coreboot/+/40662/8/Documentation/contributing/... PS8, Line 108: Grub GRUB
https://review.coreboot.org/c/coreboot/+/40662/8/Documentation/contributing/... PS8, Line 118: provide provide_s_
https://review.coreboot.org/c/coreboot/+/40662/8/Documentation/contributing/... PS8, Line 134: don't don't_s_