Merge pull request #20 from KanHarI/bugfix/deletions

Bugfix deleted files
KanHarI · Dec 10, 2022 · c9560aa · c9560aa
2 parents e1f56a6 + 4056cfa
commit c9560aa
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/lib/index.js b/lib/index.js
@@ -19014,7 +19014,9 @@ function getFilesSummaries(pullNumber, repository) {
                 commit_id: headCommitSha,
                 path: modifiedFiles[modifiedFile].filename,
                 line: Number.isFinite(modifiedFiles[modifiedFile].position)
-                    ? modifiedFiles[modifiedFile].position
+                    ? modifiedFiles[modifiedFile].position > 0
+                        ? modifiedFiles[modifiedFile].position
+                        : 1
                     : 1,
                 side: modifiedFiles[modifiedFile].position > 0 ||
                     modifiedFiles[modifiedFile].originSha === "None"

diff --git a/src/filesSummary.ts b/src/filesSummary.ts
@@ -193,7 +193,9 @@ export async function getFilesSummaries(
       commit_id: headCommitSha,
       path: modifiedFiles[modifiedFile].filename,
       line: Number.isFinite(modifiedFiles[modifiedFile].position)
-        ? modifiedFiles[modifiedFile].position
+        ? modifiedFiles[modifiedFile].position > 0
+          ? modifiedFiles[modifiedFile].position
+          : 1
         : 1,
       side:
         modifiedFiles[modifiedFile].position > 0 ||