Rename MakeBuilder to NewBuilder

And also rename MakePublisher to NewPublisher.

For more idiomatic constructor function names.

pkg/commands/publisher_test.go

@@ -60,15 +60,15 @@ func TestPublishImages(t *testing.T) {
 		bo := &options.BuildOptions{
 			ConcurrentBuilds: 1,
 		}
-		builder, err := MakeBuilder(ctx, bo)
+		builder, err := NewBuilder(ctx, bo)
 		if err != nil {
 			t.Fatalf("%s: MakeBuilder(): %v", test.description, err)
 		}
 		po := &options.PublishOptions{
 			DockerRepo:          repo,
 			PreserveImportPaths: true,
 		}
-		publisher, err := MakePublisher(po)
+		publisher, err := NewPublisher(po)
 		if err != nil {
 			t.Fatalf("%s: MakePublisher(): %v", test.description, err)
 		}

pkg/commands/resolver.go

@@ -106,8 +106,8 @@ func gobuildOptions(bo *options.BuildOptions) ([]build.Option, error) {
 	return opts, nil
 }
 
-// MakeBuilder creates a ko builder
-func MakeBuilder(ctx context.Context, bo *options.BuildOptions) (build.Interface, error) {
+// NewBuilder creates a ko builder
+func NewBuilder(ctx context.Context, bo *options.BuildOptions) (build.Interface, error) {
 	return makeBuilder(ctx, bo)
 }
 
@@ -142,8 +142,8 @@ func makeBuilder(ctx context.Context, bo *options.BuildOptions) (*build.Caching,
 	return build.NewCaching(innerBuilder)
 }
 
-// MakePublisher creates a ko publisher
-func MakePublisher(po *options.PublishOptions) (publish.Interface, error) {
+// NewPublisher creates a ko publisher
+func NewPublisher(po *options.PublishOptions) (publish.Interface, error) {
 	return makePublisher(po)
 }
 

pkg/commands/resolver_test.go

@@ -143,7 +143,7 @@ func TestMakeBuilder(t *testing.T) {
 	bo := &options.BuildOptions{
 		ConcurrentBuilds: 1,
 	}
-	builder, err := MakeBuilder(ctx, bo)
+	builder, err := NewBuilder(ctx, bo)
 	if err != nil {
 		t.Fatalf("MakeBuilder(): %v", err)
 	}
@@ -164,7 +164,7 @@ func TestMakePublisher(t *testing.T) {
 		DockerRepo:          repo,
 		PreserveImportPaths: true,
 	}
-	publisher, err := MakePublisher(po)
+	publisher, err := NewPublisher(po)
 	if err != nil {
 		t.Fatalf("MakePublisher(): %v", err)
 	}