Skip to content

Spelling errors identified by check-spelling #2608

@jsoref

Description

@jsoref

Environment details

  • OS: All
  • Node.js version: 20
  • npm version:
  • release-please version: main

Steps to reproduce

Review misspellings reported at https://github.com/jsoref/release-please/actions/runs/18634010704/attempts/1#summary-53122636570

Unrecognized words (27)
accross
APISs
bootstapper
Canididate
chnagelog
coventional
decideed
increasae
KMR
malformatted
Natve
occured
oogle
parantheses
priviledged
refrencing
resolove
rqeuest
satified
secctions
Separtor
seperator
Sfiles
straregy
tfand
triggred
withouth
Forbidden patterns

These forbidden patterns matched content:

Should be GitHub
(?<![&*.]|// |\b(?:from|import|type) )\bGithub\b(?![{()])
Write out small numbers (unless they are code)

See https://www.scribendi.com/academy/articles/when_to_spell_out_numbers_in_writing.en.html#:~:text=Writing%20Small%20and%20Large%20Numbers,ten%29%20are%20written%20as%20numerals%2e

...

Should be fall back
\bfallback(?= to)\b
Should be nonexistent
\b[Nn]o[nt][- ]existent\b
Often should be if/when (or reworded)

unless describing exceptional cases (i.e. emergencies)

(?<!just )\bin case\b(?! (?:I|s?he|they|of|we|you))
Should be ; otherwise or . Otherwise

https://study.com/learn/lesson/otherwise-in-a-sentence.html

Should be please or also, please

https://english.stackexchange.com/questions/106165/please-do-also-or-please-also

Should be rather than
\brather then\b
A fully parenthetical sentence’s period goes inside the parentheses, not outside.

https://twitter.com/nyttypos/status/1898844061873639490

Should be a
\san (?=(?:[b-df-gjkp-rtvwz]|h(?!our|s[lv]|tml|ttp|ref)|l(?!ts)|n(?!grok|pm)|s(?!sh|vg))[a-z]|x(?!ml))
Articles generally shouldn't be used without a noun and a verb
  • Perhaps you're missing a verb between the noun and the second article.
  • Or, perhaps you should remove the first verb and treat the intervening word as a verb?
  • In some cases you should add a , between the noun and the second article.
\s(?:an?|the)\s(?!wh|how\b)[A-Za-z][a-z]+[a-qs-z]\s(?:an?|the)\s
Should only be one of a, an, or the
\b(?:(?:an?|the)\s+){2,}\b
Should be an
\sa(?= (?:a(?!nd\s|s\s)|e(?!u)|i(?![ns]\s)|o(?!nc?e)|u(?!biquitous|kr|n[ai]|r[ael]|s[aeiu]|tf\d*|tili|topia|vula|v\b)|y(?!aml|args|arn|ear|oga|y)))
Should be anymore
\bany more[,.]
Should be JavaScript
\bJavascript\b
Should be TypeScript
\bTypescript\b
Should be case-(in)sensitive
\bcase (?:in|)sensitive\b
Should be greater than
\bhigher than\b
Should be into

when not phrasal and when in order to would be wrong:
https://thewritepractice.com/into-vs-in-to/

(?<!opt)\sin to\s(?!if\b)
Should probably be Otherwise,
(?<=\. )Otherwise\s
Should be or
\b(?i)true of .*false\b
Should be whether or not ...
\bwhether(?:\s\w+)+ or not\.
In English, duplicated words are generally mistakes

There are a few exceptions (e.g. "that that").
If the highlighted doubled word pair is in:

  • code, write a pattern to mask it.
  • prose, have someone read the English before you dismiss this error.

Metadata

Metadata

Assignees

Labels

priority: p3Desirable enhancement or fix. May not be included in next release.type: bugError or flaw in code with unintended results or allowing sub-optimal usage patterns.

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions