use readline instead of bufio

add readline to vendor

update CHANGELOG

CHANGELOG.md

@@ -26,6 +26,7 @@
 * Fixed a bug where the failed creation of a custom resources could cause a CloudFormation stack to fail to rollback. [#938](https://github.com/remind101/empire/pull/938)
 * Fixed a bug where waiting for a deploy to stabilize was failing if you had more than 10 services. [#944](https://github.com/remind101/empire/issues/944)
 * Fixed an issue in the Tugboat integration where the log stream to a Tugboat instance could be closed. [#950](https://github.com/remind101/empire/pull/950)
+* Fixed an issue where typing commit message does not allow user to use arrow keys, etc. [#958](https://github.com/remind101/empire/pull/958)
 
 **Performance**
 

cmd/emp/util.go

@@ -1,7 +1,6 @@
 package main
 
 import (
-	"bufio"
 	"errors"
 	"fmt"
 	"io"
@@ -15,6 +14,7 @@ import (
 	"strings"
 	"time"
 
+	"github.com/chzyer/readline"
 	"github.com/mgutz/ansi"
 	"github.com/remind101/empire/cmd/emp/hkclient"
 	"github.com/remind101/empire/pkg/heroku"
@@ -154,9 +154,11 @@ func askForMessage() (string, error) {
 	}
 
 	fmt.Println("A commit message is required, enter one below:")
-	fmt.Printf("> ")
-	reader := bufio.NewReader(os.Stdin)
-	message, err := reader.ReadString('\n')
+	reader, err := readline.New("> ")
+	if err != nil {
+		return "", err
+	}
+	message, err := reader.Readline()
 	return strings.Trim(message, " \n"), err
 }