-
Notifications
You must be signed in to change notification settings - Fork 518
Open
Labels
priority: p3Desirable enhancement or fix. May not be included in next release.Desirable 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.Error or flaw in code with unintended results or allowing sub-optimal usage patterns.
Description
Environment details
- OS: All
- Node.js version: 20
- npm version:
release-pleaseversion: 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)
...
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.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
priority: p3Desirable enhancement or fix. May not be included in next release.Desirable 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.Error or flaw in code with unintended results or allowing sub-optimal usage patterns.