fix: change the git clone url

`.git_url` seems to be of the form `git://` that doesn't seem to be valid anymore for github. This commit replaces it and uses `.clone_url` instead to clone the repo
rotki · Dec 16, 2024 · d18f79d · d18f79d
1 parent 71c4f76
commit d18f79d
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 13 deletions.
diff --git a/badges/coverage.svg b/badges/coverage.svg
diff --git a/dist/index.js b/dist/index.js
diff --git a/dist/index.js.map b/dist/index.js.map
diff --git a/src/changes.ts b/src/changes.ts
@@ -11,13 +11,10 @@ import type { ComparedDatabases, FileVersions } from './types';
 import type { PullRequest } from '@octokit/webhooks-types/schema';
 
 function cloneRepo(target: 'head' | 'base', url: string, ref: string, sha: string): string {
-  let cloneUrl = url;
+  const cloneUrl = url;
 
   validateGitUrl(cloneUrl);
 
-  if (cloneUrl.startsWith('git'))
-    cloneUrl = cloneUrl.replace('git', 'https');
-
   const repo = createDBDir(target, 'repo');
   core.info(`cloning ${cloneUrl} @ ${ref} as ${target} to ${repo}`);
   const args = ['clone', cloneUrl, '--branch', ref, '--depth', '1', repo];
@@ -82,8 +79,8 @@ export async function prepareForDiff(): Promise<FileVersions[]> {
   if (files.length === 0)
     return [];
 
-  const baseRepoUrl = base.repo.git_url;
-  const headRepoUrl = head.repo?.git_url;
+  const baseRepoUrl = base.repo.clone_url;
+  const headRepoUrl = head.repo?.clone_url;
 
   if (!headRepoUrl)
     throw new Error(`head repo url is missing ${headRepoUrl}`);