fix(issues): correctly parses multiple issues in same comment

bahmutov · Aug 17, 2016 · 0f92238 · 0f92238
1 parent 0c35084
commit 0f92238
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/src/replace-links-spec.js b/src/replace-links-spec.js
@@ -49,6 +49,13 @@ describe('replace links', () => {
     la(r === '(a360d56e, closes #11)', r);
   });
 
+  it('replaces several issue links', () => {
+    const s = 'closes [#11](https://github.com/bahmutov/manpm/issues/11),' +
+      ' and fixes [#12](https://github.com/bahmutov/manpm/issues/12)';
+    const r = replaceLinks(s);
+    la(r === 'closes #11, and fixes #12', r);
+  });
+
   it('handles dashes', () => {
     const s = 'release [17713ca1](https://github.com/bahmutov/lazy-ass/commit/17713ca1)';
     const r = replaceLinks(s);

diff --git a/src/replace-links.js b/src/replace-links.js
@@ -10,7 +10,7 @@ function replaceIssueLinks(s) {
   if (!match) {
     return s;
   }
-  return s.replace(match[0], match[1]);
+  return replaceIssueLinks(s.replace(match[0], match[1]));
 }
 
 // replace urls like