refactor: adjust if statement

Author: CH-Chang

git/git.go

@@ -60,28 +60,26 @@ func CloneRepo(ctx context.Context, logf func(string, ...any), opts CloneRepoOpt
 	if !opts.ThinPack {
 		thinPack = false
 		logf("ThinPack options is false, Marking thin-pack as unsupported")
-	} else {
-		if parsed.Hostname() == "dev.azure.com" {
-			// Azure DevOps requires capabilities multi_ack / multi_ack_detailed,
-			// which are not fully implemented and by default are included in
-			// transport.UnsupportedCapabilities.
-			//
-			// The initial clone operations require a full download of the repository,
-			// and therefore those unsupported capabilities are not as crucial, so
-			// by removing them from that list allows for the first clone to work
-			// successfully.
-			//
-			// Additional fetches will yield issues, therefore work always from a clean
-			// clone until those capabilities are fully supported.
-			//
-			// New commits and pushes against a remote worked without any issues.
-			// See: https://github.com/go-git/go-git/issues/64
-			//
-			// This is knowingly not safe to call in parallel, but it seemed
-			// like the least-janky place to add a super janky hack.
-			thinPack = false
-			logf("Workaround for Azure DevOps: marking thin-pack as unsupported")
-		}
+	} else if parsed.Hostname() == "dev.azure.com" {
+		// Azure DevOps requires capabilities multi_ack / multi_ack_detailed,
+		// which are not fully implemented and by default are included in
+		// transport.UnsupportedCapabilities.
+		//
+		// The initial clone operations require a full download of the repository,
+		// and therefore those unsupported capabilities are not as crucial, so
+		// by removing them from that list allows for the first clone to work
+		// successfully.
+		//
+		// Additional fetches will yield issues, therefore work always from a clean
+		// clone until those capabilities are fully supported.
+		//
+		// New commits and pushes against a remote worked without any issues.
+		// See: https://github.com/go-git/go-git/issues/64
+		//
+		// This is knowingly not safe to call in parallel, but it seemed
+		// like the least-janky place to add a super janky hack.
+		thinPack = false
+		logf("Workaround for Azure DevOps: marking thin-pack as unsupported")
 	}
 
 	if !thinPack {