Skip to content

Commit

Permalink
Added editorial style guide checking.
Browse files Browse the repository at this point in the history
Signed-off-by: dblock <[email protected]>
  • Loading branch information
dblock committed Oct 17, 2024
1 parent 4c48c26 commit 2ac1550
Show file tree
Hide file tree
Showing 63 changed files with 1,223 additions and 8 deletions.
75 changes: 75 additions & 0 deletions .github/vale/styles/OpenSearch/AcronymParentheses.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,75 @@
extends: conditional
message: "'%s': Spell out acronyms the first time that you use them on a page and follow them with the acronym in parentheses. Subsequently, use the acronym alone."
link: 'https://github.com/opensearch-project/documentation-website/blob/main/STYLE_GUIDE.md#acronyms'
level: warning
scope: summary
ignorecase: false

Check failure on line 6 in .github/vale/styles/OpenSearch/AcronymParentheses.yml

View workflow job for this annotation

GitHub Actions / lint

Unknown word: "ignorecase"
# Ensures that the existence of 'first' implies the existence of 'second'.
first: '\b((?<!k-)[A-Z]{1,3}\/?[A-Z]{1,3}\d{0,2}\b(?!\sCommons))'
second: '(?:\b[A-Za-z-]+ )+\(([A-Z]{1,3}\/?[A-Z]{1,3}\d{0,2})\)'
# ... with the exception of these:
exceptions:
- API
- ASCII
- AWS
- BASIC
- BM25
- CSV
- CPU

Check failure on line 18 in .github/vale/styles/OpenSearch/AcronymParentheses.yml

View workflow job for this annotation

GitHub Actions / lint

Expected sequence values to be in ascending order. 'CPU' should be before 'CSV'
- CRUD
- DNS
- DOS
- FAQ
- FTP
- GIF
- HTML
- HTTP
- HTTPS
- I/O
- ID
- IP
- JPEG
- JSON
- NAT
- NGINX
- PDF
- RAM
- REST
- RGB
- ROM
- SAML
- SDK
- SSL
- TCP
- TIFF
- TLS
- UI
- URI
- URL
- UTC
- UTF
- XML
- YAML
- CAT

Check failure on line 53 in .github/vale/styles/OpenSearch/AcronymParentheses.yml

View workflow job for this annotation

GitHub Actions / lint

Expected sequence values to be in ascending order. 'CAT' should be before 'YAML'
- GET
- PUT
- POST

Check failure on line 56 in .github/vale/styles/OpenSearch/AcronymParentheses.yml

View workflow job for this annotation

GitHub Actions / lint

Expected sequence values to be in ascending order. 'POST' should be before 'PUT'
- DELETE

Check failure on line 57 in .github/vale/styles/OpenSearch/AcronymParentheses.yml

View workflow job for this annotation

GitHub Actions / lint

Expected sequence values to be in ascending order. 'DELETE' should be before 'POST'
- AND

Check failure on line 58 in .github/vale/styles/OpenSearch/AcronymParentheses.yml

View workflow job for this annotation

GitHub Actions / lint

Expected sequence values to be in ascending order. 'AND' should be before 'DELETE'
- OR
- KB

Check failure on line 60 in .github/vale/styles/OpenSearch/AcronymParentheses.yml

View workflow job for this annotation

GitHub Actions / lint

Expected sequence values to be in ascending order. 'KB' should be before 'OR'
- MB
- GB

Check failure on line 62 in .github/vale/styles/OpenSearch/AcronymParentheses.yml

View workflow job for this annotation

GitHub Actions / lint

Expected sequence values to be in ascending order. 'GB' should be before 'MB'
- TB
- PB

Check failure on line 64 in .github/vale/styles/OpenSearch/AcronymParentheses.yml

View workflow job for this annotation

GitHub Actions / lint

Expected sequence values to be in ascending order. 'PB' should be before 'TB'
- US
- PNG

Check failure on line 66 in .github/vale/styles/OpenSearch/AcronymParentheses.yml

View workflow job for this annotation

GitHub Actions / lint

Expected sequence values to be in ascending order. 'PNG' should be before 'US'
- JVM
- N/A
- GROUP
- BY
- SELECT
- HAVING
- SQL
- TOC
- 'NULL'
6 changes: 6 additions & 0 deletions .github/vale/styles/OpenSearch/Ampersand.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
extends: existence
message: "Don't use an ampersand in place of 'and' in documentation."
nonword: true
level: warning
tokens:
- '\w +& +\w'
5 changes: 5 additions & 0 deletions .github/vale/styles/OpenSearch/Cyber.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
extends: existence
message: "Use 'cyber' as a prefix. Remove spaces or hyphens in '%s'."
level: error
tokens:
- '[Cc]yber[- ]+[a-z]*'
9 changes: 9 additions & 0 deletions .github/vale/styles/OpenSearch/DashSpacing.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,9 @@
extends: existence
message: "There should be no spaces around the dash in '%s'."
ignorecase: true
nonword: true
level: error
scope:
- ~list
tokens:
- '\w+ +-{2,3} +\w+'
22 changes: 22 additions & 0 deletions .github/vale/styles/OpenSearch/DirectionAboveBelow.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,22 @@
extends: substitution
message: "Use '%s' instead of '%s' for versions or orientation within a document. Use 'above' and 'below' only for physical space or screen descriptions."
link: 'https://github.com/opensearch-project/documentation-website/blob/main/TERMS.md'
level: warning
ignorecase: true
swap:
- 'image below': 'following image'
- 'example below': 'following example'
- 'steps below': 'following steps'
- 'section below': 'following section'
- 'table below': 'following table'
- 'image above': 'following image'
- 'example above': 'preceding example'
- 'section above': 'preceding section'
- 'table above': 'preceding table'
- 'above image': 'preceding image'
- 'above section': 'preceding section'
- 'above table': 'preceding table'
- '\d+\.\d+\s+(?:and|or)\s+above': 'later'
- '\d+\.\d+\s+(?:and|or)\s+below': 'earlier'
- 'below(?!\s+(?:the|this|\d))': 'following or later'
- 'above(?!\s+(?:the|this|\d))': 'previous, preceding, or earlier'
16 changes: 16 additions & 0 deletions .github/vale/styles/OpenSearch/DirectionTopBottom.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,16 @@
extends: substitution
message: "Use '%s' instead of '%s' for window, page, or pane references to features or controls. Use 'top' and 'bottom' only as a general screen reference."
link: 'https://github.com/opensearch-project/documentation-website/blob/main/TERMS.md'
level: warning
ignorecase: true
action:
name: replace
swap:
- top left: upper left
- bottom left: lower left
- top right: upper right
- bottom right: lower right
- top-left: upper-left
- bottom-left: lower-left
- top-right: upper-right
- bottom-right: lower-right
6 changes: 6 additions & 0 deletions .github/vale/styles/OpenSearch/Exclamation.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
extends: existence
message: "Don't use exclamation points in documentation."
nonword: true
level: error
tokens:
- '\w+!(?:\s|$)'
7 changes: 7 additions & 0 deletions .github/vale/styles/OpenSearch/FailoverNoun.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
extends: sequence
message: "Use 'failover' as an adjective or noun instead of '%s'."
level: error
ignorecase: true
tokens:
- tag: NN|JJ
pattern: '(?:fail over|fail-over)'
7 changes: 7 additions & 0 deletions .github/vale/styles/OpenSearch/FailoverVerb.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
extends: sequence
message: "Use 'fail over' as a verb instead of '%s'."
level: error
ignorecase: true
tokens:
- tag: VB|VBD|VBG|VBN|VBP|VBZ
pattern: '(?:failover|fail-over)'
7 changes: 7 additions & 0 deletions .github/vale/styles/OpenSearch/FutureTense.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
extends: existence
message: "'%s' is in future tense. Use present tense in documentation."
ignorecase: true
level: suggestion
scope: raw
tokens:
- '(?:will|is going to|won''t|[A-za-z]+''ll)\s+[a-z]+'
11 changes: 11 additions & 0 deletions .github/vale/styles/OpenSearch/HeadingAcronyms.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,11 @@
extends: existence
message: "'%s': Don't define acronyms in headings."
link: 'https://github.com/opensearch-project/documentation-website/blob/main/STYLE_GUIDE.md#acronyms'
level: error
ignorecase: false
scope: heading
nonword: true
action:
name: remove
tokens:
- '\([A-Z]{2,5}\)'
13 changes: 13 additions & 0 deletions .github/vale/styles/OpenSearch/HeadingCapitalization.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,13 @@
extends: capitalization
message: "'%s' is a heading and should be in sentence case."
level: error
scope: heading
match: $sentence
indicators:
- ":"
- "."
- ")"
exceptions:
- k # ignores lowercase k-NN
- '[A-Z]{2,}' # ignores all acronyms
- '([A-Z][a-z0-9]+){2,}' # ignores all camel case words
7 changes: 7 additions & 0 deletions .github/vale/styles/OpenSearch/HeadingColon.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
extends: existence
message: "Capitalize the word after a colon in '%s'."
nonword: true
level: error
scope: heading
tokens:
- '(?::\s)[a-z]+'
9 changes: 9 additions & 0 deletions .github/vale/styles/OpenSearch/HeadingPunctuation.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,9 @@
extends: existence
message: "Don't use punctuation at the end of a heading."
nonword: true
level: error
scope: heading
action:
name: remove
tokens:
- '[.?!]$'
15 changes: 15 additions & 0 deletions .github/vale/styles/OpenSearch/Inclusive.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,15 @@
extends: substitution
message: "Use '%s' instead of '%s' because the latter is an offensive term."
link: https://github.com/opensearch-project/documentation-website/blob/main/STYLE_GUIDE.md#offensive-terms
ignorecase: true
level: error
swap:
'abort': stop, end, or cancel
'black day': blocked day
'blacklist': deny list
'kill': stop, end, clear, remove, or cancel
'master account': 'management account'
'master': cluster manager
'slave': replica, secondary, standby
'white day': open day
'whitelist': allow list
10 changes: 10 additions & 0 deletions .github/vale/styles/OpenSearch/LatinismsElimination.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,10 @@
extends: existence
message: "Using '%s' is unnecessary. Remove."
link: https://github.com/opensearch-project/documentation-website/blob/main/STYLE_GUIDE.md#basic-guidelines
ignorecase: true
nonword: true
level: warning
action:
name: remove
tokens:
- '\b(?:etc\.|etc)'
15 changes: 15 additions & 0 deletions .github/vale/styles/OpenSearch/LatinismsSubstitution.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,15 @@
extends: substitution
message: "Use '%s' instead of '%s'."
link: https://github.com/opensearch-project/documentation-website/blob/main/STYLE_GUIDE.md#basic-guidelines
ignorecase: false
level: warning
nonword: true
action:
name: replace
swap:
'\b(?:eg|e\.g\.)[\s,]': for example or such as
'\b(?:ie|i\.e\.)[\s,]': that is or specifically
'\bad hoc[\s,.]': one-time
'\b(?:v\.|vs\.|vs|versus)\s': compared to or compared with
'\bvia\s': using, through, by accessing, or by choosing
'\bvice versa': the other way around
8 changes: 8 additions & 0 deletions .github/vale/styles/OpenSearch/LinksDoubleParentheses.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
extends: existence
message: "Remove double parentheses from the link '%s'."
level: error
nonword: true
scope: raw
tokens:
- '\]\({2,}[^)]*?\){1,}'

7 changes: 7 additions & 0 deletions .github/vale/styles/OpenSearch/LinksDoubleSlash.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
extends: existence
message: "Remove double slashes from the link '%s'."
level: error
nonword: true
scope: raw
tokens:
- '\(\{\{site.url\}\}\{\{site.baseurl\}\}[^)]*?\/{2,}[^)]*?\)'
7 changes: 7 additions & 0 deletions .github/vale/styles/OpenSearch/LinksEndSlash.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
extends: existence
message: "Add a trailing slash to the link '%s'."
level: error
nonword: true
scope: raw
tokens:
- '\(\{\{site.url\}\}\{\{site.baseurl\}\}(\/[A-Za-z0-9-_]+)+\s*\)'
7 changes: 7 additions & 0 deletions .github/vale/styles/OpenSearch/LinksMidSlash.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
extends: existence
message: "Add a slash after '{{site.url}}/{{site.baseurl}}' in '%s'."
level: error
nonword: true
scope: raw
tokens:
- '\(\{\{site.url\}\}\{\{site.baseurl\}\}([^\/])(?:(.*))?\)'
7 changes: 7 additions & 0 deletions .github/vale/styles/OpenSearch/LoginNoun.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
extends: sequence
message: "Use 'login' as an adjective or noun instead of '%s'."
level: error
ignorecase: true
tokens:
- tag: NN|JJ
pattern: '(?:log in|log-in)'
7 changes: 7 additions & 0 deletions .github/vale/styles/OpenSearch/LoginVerb.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
extends: sequence
message: "Use 'log in' as a verb instead of '%s'."
level: error
ignorecase: true
tokens:
- tag: VB|VBD|VBG|VBN|VBP|VBZ
pattern: '(?:login|log into|log on|log onto)'
7 changes: 7 additions & 0 deletions .github/vale/styles/OpenSearch/LogoutNoun.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
extends: sequence
message: "Use 'logout' as an adjective or noun instead of '%s'."
level: error
ignorecase: true
tokens:
- tag: NN|JJ
pattern: '(?:log out)'
7 changes: 7 additions & 0 deletions .github/vale/styles/OpenSearch/LogoutVerb.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
extends: sequence
message: "Use 'log out' as a verb instead of '%s'."
level: error
ignorecase: true
tokens:
- tag: VB|VBD|VBG|VBN|VBP|VBZ
pattern: '(?:logout)'
6 changes: 6 additions & 0 deletions .github/vale/styles/OpenSearch/MergeConflicts.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
extends: existence
message: "Resolve lingering merge conflicts."
nonword: true
level: error
tokens:
- '<<<<<<< HEAD'
7 changes: 7 additions & 0 deletions .github/vale/styles/OpenSearch/OxfordComma.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
extends: existence
message: "Add an Oxford comma in '%s'."
scope: sentence
level: warning
nonword: true
tokens:
- '(?:[^\s,]+,){1,} \w+ (?:and|or) \w+[.?!]'
45 changes: 45 additions & 0 deletions .github/vale/styles/OpenSearch/PassiveVoice.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,45 @@
extends: existence
message: "'%s': Whenever possible, use the active voice instead of the passive voice."
ignorecase: true
level: suggestion
raw:
- \b(am|are|were|being|is|been|was|be)\b\s*
tokens:
- '[\w]+ed'
- become
- been
- begun
- brought
- built
- cast
- caught
- chosen
- come
- cut
- dealt
- done
- drawn
- forbidden
- found
- given
- gone
- gotten
- held
- hidden
- kept
- known
- led
- let
- made
- put
- quit
- read
- seen
- sent
- sped
- spent
- stuck
- swept
- taken
- understood
- written
Loading

0 comments on commit 2ac1550

Please sign in to comment.