Skip to content

Commit

Permalink
Added editorial style guide checking. (#620)
Browse files Browse the repository at this point in the history
* Added editorial style guide checking.

Signed-off-by: dblock <[email protected]>
Signed-off-by: Theo N. Truong <[email protected]>
Co-authored-by: Theo N. Truong <[email protected]>
  • Loading branch information
dblock and nhtruong authored Oct 24, 2024
1 parent 110474f commit 61e6ec3
Show file tree
Hide file tree
Showing 61 changed files with 1,125 additions and 45 deletions.
13 changes: 13 additions & 0 deletions .cspell
Original file line number Diff line number Diff line change
Expand Up @@ -51,10 +51,14 @@ fragmenter
Fragmenter
freqs
generateonbehalfoftoken
geodistance
geohash
geohex
Geohex
geoip
geopoint
geoshape
geospatial
geotile
gsub
Gsub
Expand All @@ -65,6 +69,7 @@ homoscedastic
hotthreads
huggingface
hybridfs
ignorecase
integ
internalusers
Intertransport
Expand Down Expand Up @@ -111,6 +116,7 @@ niofs
nmslib
nodeattrs
nodesdn
nonword
noops
nori
Nori
Expand All @@ -128,7 +134,11 @@ Oversample
performanceanalyzer
permissionsinfo
pipefail
preconfigure
preconfigured
prefilter
pretrain
pretrained
prirep
rawfile
readingform
Expand Down Expand Up @@ -186,11 +196,13 @@ tfidf
Tfidf
thomaseizinger
Tika
timeframe
tlbr
tokenfilters
translog
Translog
trbl
truststore
tubone
Undeploys
unigrams
Expand All @@ -203,6 +215,7 @@ urldecode
vectory
whoamiprotected
wordnet
yber
Yrtsd
جامد
جامدات
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
# 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:
- AND
- API
- ASCII
- AWS
- BASIC
- BM25
- BY
- CAT
- CPU
- CRUD
- CSV
- DELETE
- DNS
- DOS
- FAQ
- FTP
- GB
- GET
- GIF
- GROUP
- HAVING
- HTML
- HTTP
- HTTPS
- I/O
- ID
- IP
- JPEG
- JSON
- JVM
- KB
- MB
- N/A
- NAT
- NGINX
- 'NULL'
- OR
- PB
- PDF
- PNG
- POST
- PUT
- RAM
- REST
- RGB
- ROM
- SAML
- SDK
- SELECT
- SQL
- SSL
- TB
- TCP
- TIFF
- TLS
- TOC
- UI
- URI
- URL
- US
- UTC
- UTF
- XML
- YAML
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]+'
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
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)'
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
9 changes: 9 additions & 0 deletions .github/vale/styles/OpenSearch/Please.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,9 @@
extends: existence
message: "Using '%s' is unnecessary. Remove."
link: https://github.com/opensearch-project/documentation-website/blob/main/TERMS.md
ignorecase: true
level: warning
action:
name: remove
tokens:
- please
7 changes: 7 additions & 0 deletions .github/vale/styles/OpenSearch/Range.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
extends: existence
message: Use an en dash (--) with no space on either side in a range of numbers.
link: https://github.com/opensearch-project/documentation-website/blob/main/STYLE_GUIDE.md#numbers-and-measurement
nonword: true
level: error
tokens:
- '\b\d+ *[-] *\d+\b'
6 changes: 6 additions & 0 deletions .github/vale/styles/OpenSearch/Repetition.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
extends: repetition
message: "'%s' is repeated."
level: error
alpha: true
tokens:
- '[^\s]+'
Loading

0 comments on commit 61e6ec3

Please sign in to comment.