Add ability to set messages on build creation (#32)

* Add ability to set comments on build creation

* message not comment

* add header for message field in reco build list

Author: Max Siegieda

build.go

@@ -15,13 +15,13 @@ type buildJob struct {
 	*clientImpl
 }
 
-func (b buildJob) prepareBuild() (string, error) {
+func (b buildJob) prepareBuild(message string) (string, error) {
 	projectID, err := b.projectID()
 	if err != nil {
 		return "", err
 	}
 	req := b.apiRequest(endpoints.builds.String())
-	reqBody := M{"project_id": projectID}
+	reqBody := M{"project_id": projectID, "message": message}
 	resp, err := req.Do("POST", reqBody)
 	if err != nil {
 		return "", err
@@ -39,9 +39,10 @@ func (b buildJob) prepareBuild() (string, error) {
 func (b buildJob) Start(args Args) (string, error) {
 	srcDir := String(args.At(0))
 	wait := Bool(args.At(1))
+	message := String(args.At(2))
 
 	logger.Info.Println("preparing build")
-	id, err := b.prepareBuild()
+	id, err := b.prepareBuild(message)
 	if err != nil {
 		return "", err
 	}
@@ -95,6 +96,7 @@ func (b buildJob) List(filter M) (printer.Table, error) {
 			build.Status,
 			buildTime,
 			timeRounder(build.Duration).Nearest(time.Second),
+			build.Message,
 		}
 		if allProjects {
 			row = append(row, build.Project)
@@ -104,7 +106,7 @@ func (b buildJob) List(filter M) (printer.Table, error) {
 	}
 
 	table = printer.Table{
-		Header: []string{"build id", "status", "started", "duration"},
+		Header: []string{"build id", "status", "started", "duration", "message"},
 		Body:   body,
 	}
 	if allProjects {

cmd/build.go

@@ -13,8 +13,9 @@ import (
 
 var (
 	buildVars = struct {
-		wait  bool
-		force bool
+		wait    bool
+		force   bool
+		message string
 	}{
 		wait: true,
 	}
@@ -74,6 +75,7 @@ var (
 func init() {
 	buildCmdStart.PersistentFlags().BoolVarP(&buildVars.wait, "wait", "w", buildVars.wait, "Wait for the build to complete. If wait=false, logs will only be displayed up to where the build is started and assigned its unique ID. Use 'reco build list' to check the status of your builds")
 	buildCmdStart.PersistentFlags().BoolVarP(&buildVars.force, "force", "f", buildVars.force, "Force a build to start. Ignore source code validation")
+	buildCmdStart.PersistentFlags().StringVarP(&buildVars.message, "message", "m", buildVars.message, "Add a message to describe this build's purpose")
 
 	buildCmd := genDevCommand("build", "build", "b", "builds")
 	buildCmd.AddCommand(genListSubcommand("builds", tool.Build()))
@@ -90,7 +92,7 @@ func startBuild(cmd *cobra.Command, args []string) {
 		exitWithError(errInvalidSourceDirectory)
 	}
 
-	id, err := tool.Build().Start(reco.Args{srcDir, buildVars.wait})
+	id, err := tool.Build().Start(reco.Args{srcDir, buildVars.wait, buildVars.message})
 	if err != nil {
 		exitWithError(err)
 	}

job.go

@@ -57,6 +57,7 @@ type jobInfo struct {
 	Command   string
 	Build     string
 	IPAddress string
+	Message   string
 }
 
 // UnmarshalJSON customizes JSON decoding for BuildInfo.
@@ -77,6 +78,7 @@ func (ji *jobInfo) UnmarshalJSON(b []byte) error {
 	ji.Project = str.Project.Name
 	ji.Command = str.Command
 	ji.IPAddress = str.IPAddress
+	ji.Message = str.Message
 	if str.Build.ID != "" {
 		ji.Build = str.Build.ID
 	}

request.go

@@ -174,6 +174,7 @@ type apiResponse struct {
 	} `json:"build,omitempty"`
 	// workaround for deployments
 	// TODO: fix on platform
+	Message   string  `json:"message"`
 	Events    []event `json:"events,omitempty"`
 	Command   string  `json:"command,omitempty"`
 	IPAddress string  `json:"ip_address,omitempty"`