Moves the default tag next to the type name

Author: ForestEckhardt

scribe/emitter.go

@@ -94,23 +94,29 @@ func (e Emitter) LaunchProcesses(processes []packit.Process, processEnvs ...map[
 	)
 
 	for _, process := range processes {
-		if len(process.Type) > typePadding {
-			typePadding = len([]byte(process.Type))
+		pType := process.Type
+		if process.Default {
+			pType += " " + "(default)"
+		}
+
+		if len(pType) > typePadding {
+			typePadding = len(pType)
 		}
 	}
 
 	for _, process := range processes {
-		pad := typePadding + len(process.Command) - len(process.Type)
-		p := fmt.Sprintf("%s: %*s", process.Type, pad, process.Command)
+		pType := process.Type
+		if process.Default {
+			pType += " " + "(default)"
+		}
+
+		pad := typePadding + len(process.Command) - len(pType)
+		p := fmt.Sprintf("%s: %*s", pType, pad, process.Command)
 
 		if process.Args != nil {
 			p += " " + strings.Join(process.Args, " ")
 		}
 
-		if process.Default {
-			p += " " + "(default)"
-		}
-
 		e.Subprocess(p)
 
 		// This ensures that the process environment variable is always the same no

scribe/emitter_test.go

@@ -251,7 +251,7 @@ func testEmitter(t *testing.T, context spec.G, it spec.S) {
 			Expect(buffer.String()).To(ContainLines(
 				"  Assigning launch processes:",
 				"    some-type:       some-command",
-				"    web:             web-command (default)",
+				"    web (default):   web-command",
 				"    some-other-type: some-other-command some args",
 				"",
 			))
@@ -281,7 +281,7 @@ func testEmitter(t *testing.T, context spec.G, it spec.S) {
 				Expect(buffer.String()).To(ContainLines(
 					"  Assigning launch processes:",
 					"    some-type:       some-command",
-					"    web:             web-command (default)",
+					"    web (default):   web-command",
 					`      ANOTHER_WEB_VAR -> "another-env"`,
 					`      WEB_VAR         -> "some-env"`,
 					"    some-other-type: some-other-command some args",