Skip to content

Commit c87c9ad

Browse files
committed
feat: add "if that makes sense" as a warning
include "if that makes sense" as one of the patterns that triggers the "does that make sense" warning re #144
1 parent fa495b9 commit c87c9ad

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

src/warnings/phrases.json

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -69,10 +69,11 @@
6969
"Does that make sense",
7070
"Did that make sense",
7171
"Does this make sense",
72-
"Did this make sense"
72+
"Did this make sense",
73+
"If that makes sense"
7374
],
74-
"pattern": "(does that make sense|did that make sense|does this make sense|did this make sense)",
75-
"source": "http://goop.com/how-women-undermine-themselves-with-words/",
75+
"pattern": "(does that make sense|did that make sense|does this make sense|did this make sense|if that makes sense)",
76+
"source": "https://goop.com/how-women-undermine-themselves-with-words/",
7677
"message": "\"does that make sense\" comes across either as condescending (like your audience can't understand) or it implies you feel you've been incoherent. A better way to close is something like \"I look forward to hearing your thoughts.\" You can leave it up to the other party to let you know if they are confused about something, rather than implying that you \"didn't make sense.\" --Tara Sophia Mohr"
7778
},
7879
{

0 commit comments

Comments
 (0)