Addresses PR comments

intelsdi-x · Aug 30, 2016 · f401d01 · f401d01
1 parent 335cd6c
commit f401d01
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 6 deletions.
diff --git a/control/plugin/collector_proxy_test.go b/control/plugin/collector_proxy_test.go
@@ -23,8 +23,6 @@ package plugin
 
 import (
 	"errors"
-	"log"
-	"os"
 	"testing"
 	"time"
 

diff --git a/control/plugin/session.go b/control/plugin/session.go
@@ -313,9 +313,8 @@ func init() {
 // simpleFormatter is a logrus formatter that includes only the message.
 type simpleFormatter struct{}
 
-func (_ *simpleFormatter) Format(entry *log.Entry) ([]byte, error) {
+func (*simpleFormatter) Format(entry *log.Entry) ([]byte, error) {
 	b := &bytes.Buffer{}
-	fmt.Fprintf(b, "%s", entry.Message)
-	b.WriteByte('\n')
+	fmt.Fprintf(b, "%s\n", entry.Message)
 	return b.Bytes(), nil
 }
diff --git a/scheduler/workflow.go b/scheduler/workflow.go
@@ -21,7 +21,6 @@ package scheduler
 
 import (
 	"errors"
-	"fmt"
 	"sync"
 
 	log "github.com/Sirupsen/logrus"