Patch Set 1:

The 'c' in coreboot is actually intended to be lowercase, as "coreboot" is the trademark (https://trademarks.justia.com/869/60/coreboot-86960637.html).

The reason I submitted the change was to make it easier to read - while I was reading the documentation, I had to reparse the sentence, to find it had ended.

As far as I am concerned, sentences end with a period.

If the lowercase spelling of "coreboot" at the beginning of a sentence is so much of an issue that you can't live with it, you might try to rewrite the affected sentences (there are waaay more than just one) so that they don't start with "coreboot". However, the readability will suffer a lot more.

Patch set 1:Code-Review -1

View Change

1 comment:

To view, visit change 39088. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ia8fa16017043b2a345c42f9b6f004df3e71c2f7b
Gerrit-Change-Number: 39088
Gerrit-PatchSet: 1
Gerrit-Owner: Ivan Labáth <iger@labo.rs>
Gerrit-Reviewer: Angel Pons <th3fanbus@gmail.com>
Gerrit-Reviewer: David Hendricks <david.hendricks@gmail.com>
Gerrit-Reviewer: HAOUAS Elyes <ehaouas@noos.fr>
Gerrit-Reviewer: Ivan Labáth <iger@labo.rs>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter@users.sourceforge.net>
Gerrit-Comment-Date: Mon, 24 Feb 2020 15:32:06 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment