Skip to content

Commit b6e657f

Browse files
m4sterbunnyjoshuafernandes
authored andcommitted
fixes for vale
1 parent 1acd6aa commit b6e657f

File tree

11 files changed

+747
-99
lines changed

11 files changed

+747
-99
lines changed

spelling/.vale.ini

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,8 +5,11 @@
55

66
StylesPath = ./styles
77

8+
# The ignore config references project-words.txt
9+
810
Spelling.Ignore = config/ignore/Consensys-common
911

12+
1013
# Packages to sync with. Remember never adapt the external packages, all changes will be lost on update
1114

1215
Packages = Microsoft, proselint, write-good
@@ -15,10 +18,11 @@ Vocab = Consensys-common
1518

1619
# The minimum alert level to display (suggestion, warning, or error).
1720
# Builds not set to fail too many false positives
18-
MinAlertLevel = suggestion
21+
MinAlertLevel = warning
1922

2023
[formats]
2124
mdx = md
25+
yaml = yml
2226

2327
# Global settings (applied to every syntax)
2428
[*.{md,yml}]

spelling/README.md

Lines changed: 7 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -39,21 +39,19 @@ be compiled to this one sot.
3939

4040
If API specs are managed as YAML files, need to test whether Vale can handle linting YAMLs.
4141

42-
## Testing Vale Here
43-
44-
This heading should trigger warning for non-compliance with sentence case. Metamask should upset
45-
Vale too.
46-
4742
## ToDo
4843

44+
### Incorporate project-words.txt from repos
45+
[x] Besu
46+
[x] Teku
47+
4948
### Overrides
5049

5150
Attempted to overcome false positives with:
5251

53-
1. /styles/ignore which references the Consensys-common/ignore.txt text file (which should override Vale.Spelling)
54-
2. With styles/Consensys/Overridesyaml which is a dummy extend of "existence" to allow use of an exceptions
55-
list {I am not convinced this is working and its not a ssot method as it will need to repeat the ignore items
56-
for Vale.Spelling}
52+
1. /styles/ignore which references the Consensys-common/ignore.txt text file
53+
2. With styles/Consensys/ProjectWords.yml which is a dummy extend of "existence" to allow use of an exceptions
54+
list
5755

5856

5957

spelling/styles/Consensys/Acronyms.yml

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@ exceptions:
1515
- CLI
1616
- CORS
1717
- CPU
18+
- CSS
1819
- CSV
1920
- DEBUG
2021
- ECDSA
@@ -26,6 +27,7 @@ exceptions:
2627
- GET
2728
- GPU
2829
- gRPC
30+
- GTag
2931
- GUI
3032
- HEX
3133
- HTML
@@ -40,13 +42,14 @@ exceptions:
4042
- JDK
4143
- JRE
4244
- JSON
45+
- JSX
4346
- JVM
4447
- KMS
4548
- P2P
4649
- PATH
4750
- PBFT
4851
- PKCS
49-
- POA
52+
- PoA
5053
- POS
5154
- POST
5255
- PostgreSQL
@@ -63,6 +66,7 @@ exceptions:
6366
- SSL
6467
- SVG
6568
- TCP
69+
- TKBU
6670
- TLS
6771
- TOC
6872
- TOML

spelling/styles/Consensys/CaseSensitive-Substitution.yaml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
extends: substitution
2-
message: For system-specific term, consider %s instead of "%s"
2+
message: For component of a system or named entity, consider standard format, %s instead of "%s"
33
ignorecase: false
44
level: warn
55
action:
@@ -11,3 +11,5 @@ swap:
1111
prover: Prover
1212
# Linea Voyage
1313
voyage: Voyage
14+
besu: Besu
15+
Dapp: dapp

spelling/styles/Consensys/Overrides.yml

Lines changed: 0 additions & 24 deletions
This file was deleted.
Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,9 @@
1+
extends: substitution
2+
message: "Prefer Consensys style."
3+
level: error
4+
ignorecase: true
5+
action:
6+
name: replace
7+
swap:
8+
: "'"
9+
: '"'

spelling/styles/config/ignore/Consensys-common/ignore.txt

Lines changed: 0 additions & 55 deletions
This file was deleted.

0 commit comments

Comments
 (0)