-
Notifications
You must be signed in to change notification settings - Fork 361
Standardize the capitalization of specific words #2911
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
docs/books/admin_guide/12-network.md
Outdated
| * Upper Layer Protocol Data Unit | ||
| In some books, the Extended Header and Upper Layer Protocol Data Unit are collectively referred to as the **Payload**. | ||
| In some books, the Extendsion Header and Upper Layer Protocol Data Unit are collectively referred to as the **Payload**. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
typo(?). please fix
|
Hey @jimcat8 and @wsoyinka I'm not a fan of adding capital's to things like "Source address." I'm leaving this decision up to @wsoyinka, but in my view most of the proposed changes here are incorrect both grammatically and technically. Again, I'm leaving the final decision on this to @wsoyinka and will let him decide. I can tell you that if you use an editor that is technically aware, it will not capitalize most of the changes made here, and in fact will call them errors. |
Applied @sspencerwire feedback. Thank you @jimcat8 !
Test results for cfd0b44:
|
Author checklist (Completed by original Author)
Rocky Documentation checklist (Completed by Rocky team)