fix(git-resolver): resolve commits via https

pnpm · May 14, 2022 · eb7293d · eb7293d
1 parent 4985aab
commit eb7293d
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/.changeset/violet-kids-clean.md b/.changeset/violet-kids-clean.md
@@ -0,0 +1,5 @@
+---
+"@pnpm/git-resolver": patch
+---
+
+Resolve commits from GitHub via https.
diff --git a/packages/git-resolver/src/parsePref.ts b/packages/git-resolver/src/parsePref.ts
@@ -77,10 +77,7 @@ function urlToFetchSpec (urlparse: URL) {
 async function fromHostedGit (hosted: any): Promise<HostedPackageSpec> { // eslint-disable-line
   let fetchSpec: string | null = null
   // try git/https url before fallback to ssh url
-
-  // Note only GitHub has gitTemplate (git://...) in hosted-git-info, has to use
-  // sshTemplate (git@...) for bitbucket and gitlab.
-  const gitUrl = hosted.git({ noCommittish: true }) ?? hosted.ssh({ noCommittish: true })
+  const gitUrl = hosted.https({ noCommittish: true }) ?? hosted.ssh({ noCommittish: true })
   if (gitUrl && await accessRepository(gitUrl)) {
     fetchSpec = gitUrl
   }