todo-backend: use integrations to resolve line numbers

Signed-off-by: Patrik Oldsberg <poldsberg@gmail.com>
2021-03-10 22:35:31 +01:00
parent bdf5d7de59
commit 4fb393e98f
1 changed files with 5 additions and 6 deletions
@@ -114,10 +114,6 @@ export class TodoScmReader implements TodoReader {
          path: file.path,
          content: content.toString('utf8'),
        });
-        const viewUrl = this.integrations.resolveUrl({
-          url: file.path,
-          base: url,
-        });

        todos.push(
          ...items.map(({ lineNumber, text, author }) => ({
@@ -125,8 +121,11 @@ export class TodoScmReader implements TodoReader {
            author,
            lineNumber,
            repoFilePath: file.path,
-            viewUrl:
-              lineNumber === undefined ? viewUrl : `${viewUrl}#L${lineNumber}`,
+            viewUrl: this.integrations.resolveUrl({
+              url: file.path,
+              base: url,
+              lineNumber,
+            }),
          })),
        );
      } catch (error) {