feat: support adding people with mentions (#36)

* feat: support adding people with mentions * Tweak * Improve
all-contributors · Jan 15, 2019 · 5f79140 · 5f79140
1 parent f104a0a
commit 5f79140
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 1 deletion.
diff --git a/src/utils/parse-comment/index.js b/src/utils/parse-comment/index.js
@@ -57,7 +57,7 @@ const plugin = {
 nlp.plugin(plugin)
 
 function parseAddComment(doc, action) {
-    const who = doc
+    const whoMatched = doc
         .match(`${action} [.]`)
         .normalize({
             whitespace: true, // remove hyphens, newlines, and force one space between words
@@ -75,6 +75,8 @@ function parseAddComment(doc, action) {
         })
         .data()[0].text
 
+    const who = whoMatched.startsWith('@') ? whoMatched.substr(1) : whoMatched
+
     // TODO: handle plurals (e.g. some said docs)
     let contributions = doc
         .match('#Contribution')

diff --git a/test/utils/parse-comment/index.test.js b/test/utils/parse-comment/index.test.js
@@ -55,6 +55,18 @@ describe('parseComment', () => {
         })
     })
 
+    test('Support adding people with mentions', () => {
+        expect(
+            parseComment(
+                `@${testBotName} please add @sinchang for infrastructure`,
+            ),
+        ).toEqual({
+            action: 'add',
+            who: 'sinchang',
+            contributions: ['infra'],
+        })
+    })
+
     // TODO: make it so this works
     //     test('Support split words (like user testing)', () => {
     //         expect(