-
Notifications
You must be signed in to change notification settings - Fork 4
Description
Context: I've been testing these rules against existing docs to identify problematic patterns that we might be able to fix before a larger rollout.
There are probably some acronyms (or words that are in ALL CAPS that are currently being interpreted as acronyms by the linter) that we should add to the list of exceptions in Acronyms.yml. Here are the results: acronyms (summary).
It looks like in 9312937 (I think this was committed to main without a PR) added most (all?) of the acronyms found in the existing docs. However, I don't think we want to add all of them to the list of exceptions because then there's not really any value to the acronym rule.
Writers should look critically at the list of existing acronyms to:
- Identify acronyms that we're comfortable assuming readers already know and add them to the list of exceptions.
- Identify words that are in ALL CAPS that are currently being interpreted as acronyms and should be allowed to continue to be in ALL CAPS without definition. Then add them to the list of exceptions.
In my opinion (but you may disagree with me!) the benefits of this approach justify the time and effort it would take to review the list of acronyms critically. The benefits include bulk-updating the list of exceptions ahead of a larger rollout so writers don't have to add as many one-off exceptions later and reducing noise for developers after the rollout (which IMO will help us build goodwill across the engineering org or at least not lose any :)).
cc @theletterf