Fix FirstPerson.yml

We can't rely on word boundaries for finding instances of "I" because things like "i.e." will trigger matches.
errata-ai · Mar 30, 2019 · aae6929 · aae6929
1 parent e069db8
commit aae6929
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/Microsoft/FirstPerson.yml b/Microsoft/FirstPerson.yml
@@ -3,6 +3,6 @@ message: "Use first person sparingly ('%s')."
 link: 'https://docs.microsoft.com/en-us/style-guide/grammar/person'
 ignorecase: true
 level: warning
+nonword: true
 tokens:
-  - I
-  - me
+  - (?:I\s|\bme\b)
diff --git a/fixtures/Terms/test.md b/fixtures/Terms/test.md
@@ -10,4 +10,6 @@ I did that too. There was adequate number of people.
 
 Did you ensure that it works?
 
+This is a word -- i.e., something else.
+
Original file line number	Diff line number	Diff line change
Expand Up		@@ -10,4 +10,6 @@ I did that too. There was adequate number of people.

		Did you ensure that it works?

		This is a word -- i.e., something else.