Skip to content

docs: fix English spelling and grammar across pattern documentation#47

Open
Technocrat-dev wants to merge 1 commit intomercari:masterfrom
Technocrat-dev:docs/fix-english-grammar
Open

docs: fix English spelling and grammar across pattern documentation#47
Technocrat-dev wants to merge 1 commit intomercari:masterfrom
Technocrat-dev:docs/fix-english-grammar

Conversation

@Technocrat-dev
Copy link

What this PR does / why we need it:

Fixes English spelling and grammar errors across 12 pattern documentation files. Found these while reading through the patterns.

Fixes #7

Changes:

Serving patterns:

  • Asynchronous: "poling" -> "polling", "predition" -> "prediction", "disposes" -> "discard"
  • Circuit-break: "take a file" -> "take a while", "respond the" -> "respond to the"
  • Web-single: "it's" -> "its" (possessive)
  • Prep-pred: "binalized" -> "binarized"
  • Microservice-horizontal: "prediciton" -> "prediction", "parallelly" -> "in parallel"
  • Serving-template: "smodel" -> "model", "choise" -> "choice", "paramenter" -> "parameter"
  • Multiple-stage: "fare" -> "fair"

QA patterns:

  • Shadow AB test: "verity" -> "verify" (x2), "the a" -> "that a"
  • Online AB test: "verity" -> "verify" (x2), "the a" -> "that a", "parallelly" -> "in parallel"
  • Loading test: "test" -> "text" (in data type list)

Operation patterns:

  • Prediction-monitoring: Added missing # in markdown header
  • Data-model-versioning: "differenct" -> "different", "fuction" -> "function", "paramater" -> "parameter"

Special notes for your reviewer:

All changes are spelling/grammar only, no content changes. This is my first contribution to this project.

- Fix typos: poling, predition, prediciton, binalized, verity, differenct, fuction, paramater, paramenter, smodel, choise
- Fix grammar: 'take a file' to 'take a while', 'it's' to 'its' (possessive), 'the a' to 'that a', 'parallelly' to 'in parallel', 'fare' to 'fair'
- Fix missing markdown header in Prediction monitoring pattern
- Fix 'test' to 'text' in Loading test pattern data type list
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

English grammer check

1 participant