manifests: refactor: PlainCloneContext() -> PlainClone()

This is because we don't seem to use the context in any meaningful way here.

manifests: refactor: PlainCloneContext() -> PlainClone()
This is because we don't seem to use the context in any meaningful way here.
ffa116ca · Michał Flendrich · 7a826885 · ffa116ca
Commit ffa116ca authored 5 years ago by Michał Flendrich
Hide whitespace changes
Inline Side-by-side

Showing

with 1 addition and 3 deletions
+1 -3
--- a/pkg/manifests/manifests.go
+++ b/pkg/manifests/manifests.go
 package manifests

 import (
-	"context"
 	"io/ioutil"
 	"os"
 	"path/filepath"
@@ -34,9 +33,8 @@ func syncRepo(url, branch, deployKeyPath, relativeRoot string) (string, string,
 		os.RemoveAll(srcDir)
 	}
 	lCtx := log.WithField("repo", url)
-	ctx := context.Background()
 	opt := cloneOptions(url, deployKeyPath, branch)
-	r, err := gogit.PlainCloneContext(ctx, srcDir, false, &opt)
+	r, err := gogit.PlainClone(srcDir, false, &opt)

 	if err != nil {
 		closer()