DEV: Minor readability improvements to launcher2 (#843)

Author: tgxworld

launcher_go/v2/cli_build_test.go

@@ -6,13 +6,14 @@ import (
 
 	"bytes"
 	"context"
-	ddocker "github.com/discourse/discourse_docker/launcher_go/v2"
-	. "github.com/discourse/discourse_docker/launcher_go/v2/test_utils"
-	"github.com/discourse/discourse_docker/launcher_go/v2/utils"
 	"io"
 	"os"
 	"os/exec"
 	"strings"
+
+	ddocker "github.com/discourse/discourse_docker/launcher_go/v2"
+	. "github.com/discourse/discourse_docker/launcher_go/v2/test_utils"
+	"github.com/discourse/discourse_docker/launcher_go/v2/utils"
 )
 
 var _ = Describe("Build", func() {

launcher_go/v2/docker/commands.go

@@ -3,17 +3,18 @@ package docker
 import (
 	"context"
 	"fmt"
-	"github.com/Wing924/shellwords"
-	"github.com/discourse/discourse_docker/launcher_go/v2/config"
-	"github.com/discourse/discourse_docker/launcher_go/v2/utils"
-	"golang.org/x/sys/unix"
 	"io"
 	"os"
 	"os/exec"
 	"runtime"
 	"strings"
 	"syscall"
 	"time"
+
+	"github.com/Wing924/shellwords"
+	"github.com/discourse/discourse_docker/launcher_go/v2/config"
+	"github.com/discourse/discourse_docker/launcher_go/v2/utils"
+	"golang.org/x/sys/unix"
 )
 
 type DockerBuilder struct {
@@ -91,6 +92,7 @@ func (r *DockerRunner) Run() error {
 			return unix.Kill(-cmd.Process.Pid, unix.SIGINT)
 		}
 	}
+
 	cmd.Env = r.Config.EnvArray(true)
 
 	if r.DryRun {
@@ -113,54 +115,66 @@ func (r *DockerRunner) Run() error {
 		cmd.Args = append(cmd.Args, "--env")
 		cmd.Args = append(cmd.Args, e)
 	}
+
 	for k, v := range r.Config.Labels {
 		cmd.Args = append(cmd.Args, "--label")
 		cmd.Args = append(cmd.Args, k+"="+v)
 	}
+
 	if !r.SkipPorts {
 		for _, v := range r.Config.Expose {
 			if strings.Contains(v, ":") {
-				cmd.Args = append(cmd.Args, "-p")
+				cmd.Args = append(cmd.Args, "--publish")
 				cmd.Args = append(cmd.Args, v)
 			} else {
 				cmd.Args = append(cmd.Args, "--expose")
 				cmd.Args = append(cmd.Args, v)
 			}
 		}
 	}
+
 	for _, v := range r.Config.Volumes {
-		cmd.Args = append(cmd.Args, "-v")
+		cmd.Args = append(cmd.Args, "--volume")
 		cmd.Args = append(cmd.Args, v.Volume.Host+":"+v.Volume.Guest)
 	}
+
 	for _, v := range r.Config.Links {
 		cmd.Args = append(cmd.Args, "--link")
 		cmd.Args = append(cmd.Args, v.Link.Name+":"+v.Link.Alias)
 	}
+
 	cmd.Args = append(cmd.Args, "--shm-size=512m")
+
 	if r.Rm {
 		cmd.Args = append(cmd.Args, "--rm")
 	}
+
 	if r.Restart {
 		cmd.Args = append(cmd.Args, "--restart=always")
 	} else {
 		cmd.Args = append(cmd.Args, "--restart=no")
 	}
+
 	if r.Detatch {
 		cmd.Args = append(cmd.Args, "-d")
 	}
+
 	cmd.Args = append(cmd.Args, "-i")
 
 	// Docker args override settings above
 	for _, f := range r.Config.DockerArgs() {
 		cmd.Args = append(cmd.Args, f)
 	}
+
 	for _, f := range r.ExtraFlags {
 		cmd.Args = append(cmd.Args, f)
 	}
+
 	cmd.Args = append(cmd.Args, "-h")
 	cmd.Args = append(cmd.Args, r.Hostname)
 	cmd.Args = append(cmd.Args, "--name")
 	cmd.Args = append(cmd.Args, r.ContainerId)
+
 	if len(r.CustomImage) > 0 {
 		cmd.Args = append(cmd.Args, r.CustomImage)
 	} else {
@@ -176,7 +190,9 @@ func (r *DockerRunner) Run() error {
 		cmd.Stderr = os.Stderr
 		cmd.Stdin = r.Stdin
 	}
+
 	runner := utils.CmdRunner(cmd)
+
 	if r.DryRun {
 		fmt.Println(cmd)
 	} else {