Skip to content
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

Edit documentation for consistency #27895

Open
11 of 16 tasks
philwebb opened this issue Sep 9, 2021 · 5 comments
Open
11 of 16 tasks

Edit documentation for consistency #27895

philwebb opened this issue Sep 9, 2021 · 5 comments
Labels
type: epic An issue tracking a large piece of work that will be split into smaller issues
Milestone

Comments

@philwebb
Copy link
Member

philwebb commented Sep 9, 2021

With #27759 @Buzzardo applied an editing pass to the actuator documentation. There are quite a few patterns that emerged from that set that we should try and adopt going forward. We can update this epic with items as we identify them and create distinct issues/commits for each one when we apply it.

@philwebb philwebb added the type: epic An issue tracking a large piece of work that will be split into smaller issues label Sep 9, 2021
@philwebb philwebb added this to the 2.x milestone Sep 9, 2021
@Buzzardo
Copy link

Buzzardo commented Sep 9, 2021

In a couple of cases, you have identified specific examples of wider rules. Expanding "don't" to "do not" and "it's to "it is" are instances of the broader rule to not use contractions. That is, an apostrophe should always indicate a possessive. (That one comes from formal academic writing, which has had a large influence on technical writing over the years.)

"e.g." to "for example" and "i.e." to "that is" and "via" to "through" are all examples of avoiding Latin. More than half the population confuse "e.g." and "i.e.". Similarly, most people couldn't tell you what "via" actually means. (It's Latin for "through." I studied Latin in college.) Also, screen readers sometimes do awful things to them, most notably rendering "e.g." as "egg."

"Will enable" is an instance of the broader rule to never use "will" or otherwise use the future tense. The better reason to do that is consistency. Keeping everything in the simple present tense is more consistent. Doing so often makes sentences shorter, too. The worse reason is one of those funny-sad stories. When I worked at GE, we got sued because we had used "will" in a document. Some twit sued us because we made a promise we didn't keep. The case got tossed out, but GE had to pay lawyers to deal with it.

Aside from consistency, all three of these editing rules have another reason behind them, too: English as a second language. Consistency is itself a huge help to people who may not have a strong grasp of English and have to read things multiple times to get the meaning. Avoiding Latin (and other languages that aren't English) keeps yet another language out of the mix. Expanding contractions avoids potential misreadings. Keeping the tense in the simple present avoids parsing of more than one tense.

Finally, many cultures expect a greater level of formality than native English speakers expect. If our audience were solely native English speakers, we could be a lot more "loose" in our writing. However, the Chinese (especially) and other asian cultures expect formal structures in their technical writing. To them, it is official writing, and they believe it should be written that way. They are more comfortable with more formal writing, and we should meet them where they are, so long as doing so does not cause other troubles (and all these other rules have other good reasons behind them, too).

@snicoll
Copy link
Member

snicoll commented Sep 21, 2021

While reviewing #28064, I was wondering if "Let's" should be expanded to "Let us". I wasn't sure so I didn't apply that.

@Buzzardo
Copy link

"Let's" should always be expanded. It is a contraction of "Let us".

@HiuKwok
Copy link

HiuKwok commented Sep 24, 2023

I did a global search on codebase, and I only see one occurance of deny, and I don't think that makes sense to swap it with prevent, hence we can probably mark We should when possible change "deny" to "prevent". as done.

@wilkinsona
Copy link
Member

Thanks, @HiuKwok. I've done that.

@philwebb philwebb modified the milestones: 2.7.x, 3.3.x Nov 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: epic An issue tracking a large piece of work that will be split into smaller issues
Projects
None yet
Development

No branches or pull requests

5 participants