Change in coreboot[master]: Documentation/flash_tutorial: Capitalize start of proper sentence
Ivan Labáth has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/39088 )
Change subject: Documentation/flash_tutorial: Capitalize start of proper sentence ......................................................................
Documentation/flash_tutorial: Capitalize start of proper sentence
Change-Id: Ia8fa16017043b2a345c42f9b6f004df3e71c2f7b Signed-off-by: Ivan Labáth iger@labo.rs --- M Documentation/flash_tutorial/index.md 1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/88/39088/1
diff --git a/Documentation/flash_tutorial/index.md b/Documentation/flash_tutorial/index.md index 4338297..06b4033 100644 --- a/Documentation/flash_tutorial/index.md +++ b/Documentation/flash_tutorial/index.md @@ -28,7 +28,7 @@ aren't locked down.
There are various protection schemes that make it impossible to modify or -replace a firmware from a running system. coreboot allows to disable these +replace a firmware from a running system. Coreboot allows to disable these mechanisms, making it possible to overwrite (or update) the firmware from a running system.
David Hendricks has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39088 )
Change subject: Documentation/flash_tutorial: Capitalize start of proper sentence ......................................................................
Patch Set 1: Code-Review-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).
Ivan Labáth has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39088 )
Change subject: Documentation/flash_tutorial: Capitalize start of proper sentence ......................................................................
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.
IANAL, but it doesn't seem reasonable lowercase 'c' would be needed for trademark purposes, more likely a consistent branding preference. If it is what you prefer, it is your prerogative.
References:
Adidas has a lowercase trademark and seems almost consistent on not capitalizing its name - often by using all lowercase or all uppercase throughout: https://trademarks.justia.com/search?q=adidas https://www.adidas.com/us/yeezy Web search shows capitalized name on their site, but I had to view source to actually see it on a page: view-source:https://www.adidas.com/us/yeezy Secondary sources like the press and wikipedia don't seem to care about Adidas lowercase preference, at all: https://www.bbc.co.uk/news/av/uk-england-lancashire-49951277/adidas-trainers... https://www.independent.co.uk/topic/Adidas https://en.wikipedia.org/wiki/Adidas
AFAICT Intel has a lowercase trademark, but doesn't seem to mind capitalization: https://trademarks.justia.com/search?q=intel https://www.intel.com/content/www/us/en/company-overview/company-overview.ht... https://ark.intel.com/content/www/us/en/ark/products/series/186673/9th-gener...
HAOUAS Elyes has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39088 )
Change subject: Documentation/flash_tutorial: Capitalize start of proper sentence ......................................................................
Patch Set 1: Code-Review-1
Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39088 )
Change subject: Documentation/flash_tutorial: Capitalize start of proper sentence ......................................................................
Patch Set 1:
Ivan, thank you for your patch. How to write names at the beginning of sentences is a subjective decision, and the coreboot project decided to always write it lowercase. It might differ is some regions/languages and writing style guides, but I believe there are no hard rules, and it’s allowed. I’d agree with you in this, but it’s how it was decided.
Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39088 )
Change subject: Documentation/flash_tutorial: Capitalize start of proper sentence ......................................................................
Patch Set 1: Code-Review-1
(1 comment)
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.
https://review.coreboot.org/c/coreboot/+/39088/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/39088/1//COMMIT_MSG@7 PS1, Line 7: proper sentence What's a "proper sentence"?
Ivan Labáth has abandoned this change. ( https://review.coreboot.org/c/coreboot/+/39088 )
Change subject: Documentation/flash_tutorial: Capitalize start of proper sentence ......................................................................
Abandoned
avoid bikeshedding
-
Angel Pons (Code Review) -
David Hendricks (Code Review)
-
HAOUAS Elyes (Code Review)
-
Ivan Labáth (Code Review)
-
Paul Menzel (Code Review)