Skip to content
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
23 changes: 23 additions & 0 deletions .github/workflows/spelling.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,23 @@
---
name: Spell check

on:
pull_request:
push:
branches:
- main
workflow_dispatch:

permissions:
contents: read

jobs:
vale:
name: Vale
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v5
with:
fetch-depth: 0
- name: Run Vale
uses: ./docs-spelling-check
5 changes: 4 additions & 1 deletion .gitignore
Original file line number Diff line number Diff line change
@@ -1,2 +1,5 @@
.DS_Store
.idea
.idea
docs-spelling-check/styles/Microsoft/
docs-spelling-check/styles/proselint/
docs-spelling-check/styles/write-good/
3 changes: 3 additions & 0 deletions docs-spelling-check/.vale.ini
Original file line number Diff line number Diff line change
Expand Up @@ -82,6 +82,9 @@ Microsoft.Contractions = NO
# Microsoft.GeneralURL is not relevant
Microsoft.GeneralURL = NO

# Allow "agent" variants despite Microsoft.Terms preference
Microsoft.Terms.Ignore = (?i)\bagents?\b
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Microsoft.Terms.Ignore may not be valid Vale config

Medium Severity

Microsoft.Terms.Ignore does not appear to be a documented Vale configuration key. The .Ignore suffix is supported for the built-in Spelling rule (to reference dictionary files), but Microsoft.Terms is a substitution-type rule which only supports exceptions and vocab in its YAML definition. Vale will likely silently ignore this setting, meaning "agent" variants won't actually be excluded from Microsoft.Terms checks. The vocabulary accept.txt or the rule's exceptions array would be the standard way to achieve this.

Fix in Cursor Fix in Web


# Allows Microsoft.Adverbs to take precedence
Microsoft.Very = NO

Expand Down
1 change: 1 addition & 0 deletions docs-spelling-check/styles/Consensys/Headings.yml
Original file line number Diff line number Diff line change
Expand Up @@ -42,6 +42,7 @@ exceptions:
- Ethers.js
- EthSigner
- EVM
- Embedded Wallets
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Heading exception breaks alphabetical list ordering

Low Severity

Embedded Wallets is appended after EVM but the exceptions list is alphabetically ordered. Since "Em" sorts before "EO"/"ER"/"ET"/"EV", this entry belongs between Electron and Emmet to maintain the established ordering convention.

Fix in Cursor Fix in Web

- FAQs
- Flask
- Go
Expand Down
6 changes: 0 additions & 6 deletions docs-spelling-check/styles/Microsoft/Very.yml

This file was deleted.

Original file line number Diff line number Diff line change
Expand Up @@ -94,6 +94,7 @@ InfluxDB
Infura
[iI]nterlockUrl
IntraLayer
[iI]nviter(?:'s)?
Irys
Jasypt
JMeter
Expand Down Expand Up @@ -158,6 +159,7 @@ paramStructure
[pP]asswordless
Pegasys
[pP]ermission(?:ing|ed)?
[pP]imlico(?:'s)?
[pP]repend(?:s|ing|ed)?
[Pp]recompile[s]?
Photoshop
Expand Down Expand Up @@ -242,6 +244,7 @@ USB Armory Mk II
Vercel
Vertx
vCPU[s]?
[vV]iem(?:'s)?
[vV]ue
[wW]agmi
[wW]eb3
Expand Down
Loading