Fix Google.Spacing

See #6. Signed-off-by: Joseph Kato <[email protected]>
errata-ai · Jan 25, 2021 · 064d3cf · 064d3cf
1 parent 41884f7
commit 064d3cf
Show file tree

Hide file tree

Showing 4 changed files with 12 additions and 3 deletions.
diff --git a/Google/Spacing.yml b/Google/Spacing.yml
@@ -4,5 +4,5 @@ link: 'https://developers.google.com/style/sentence-spacing'
 level: error
 nonword: true
 tokens:
-  - '[.?!] {2,}[A-Z]'
-  - '[.?!][A-Z]'
+  - '[a-z][.?!] {2,}[A-Z]'
+  - '[a-z][.?!][A-Z]'
diff --git a/features/rules.feature b/features/rules.feature
@@ -32,7 +32,7 @@ Feature: Rules
       test.md:29:21:Google.WordList:Use 'preceding' instead of 'above'.
       test.md:31:1:Google.FirstPerson:Avoid first-person pronouns such as 'I '.
       test.md:31:8:Google.FirstPerson:Avoid first-person pronouns such as 'my'.
-      test.md:31:11:Google.WordList:Use 'phone or mobile phone' instead of 'cell phone'.
+      test.md:31:11:Google.WordList:Use 'phone' or 'mobile phone' instead of 'cell phone'.
       """
 
   Scenario: Use of contractions
@@ -98,6 +98,8 @@ Feature: Rules
       test.md:27:44:Google.Exclamation:Don't use exclamation points in text.
       test.md:29:1:Google.OxfordComma:Use the Oxford comma in 'Apples, pears or'.
       test.md:33:5:Google.Exclamation:Don't use exclamation points in text.
+      test.md:37:21:Google.Spacing:'y. T' should have one space.
+      test.md:39:42:Google.Exclamation:Don't use exclamation points in text.
       """
 
   Scenario: Foreign words

diff --git a/fixtures/Punctuation/.vale.ini b/fixtures/Punctuation/.vale.ini
@@ -15,3 +15,4 @@ Google.LyHyphens = YES
 Google.Parens = YES
 Google.Quotes = YES
 Google.Semicolons = YES
+Google.Spacing = YES
diff --git a/fixtures/Punctuation/test.md b/fixtures/Punctuation/test.md
@@ -31,3 +31,9 @@ Apples, pears or bananas
 ![An image](../my-image.png)
 
 Hello!
+
+this is a sentence. so is this one.
+
+This sentence is okay.  This one is not.
+
+The latest version of .NET Core is awesome!