refactoring package

Makefile

@@ -9,7 +9,7 @@ test:
 	go test $$(go list ./... | tr '\n' ' ')
 
 build:
-	go build -o ./bin/kangol *.go
+	go build -o ./bin/kangol cmd/kangol/main.go
 
 update:
 	go get -u=patch

cmd/kangol/main.go

@@ -0,0 +1,17 @@
+package main
+
+import (
+	log "github.com/Sirupsen/logrus"
+	"github.com/recruit-mp/kangol/internal"
+	"os"
+)
+
+const version = "0.2.8"
+
+func main() {
+	app := internal.NewApp(version)
+	err := app.Run(os.Args)
+	if err != nil {
+		log.Fatal(err.Error())
+	}
+}

main.go → internal/app.go

@@ -1,18 +1,17 @@
-package main
+package internal
 
 import (
-	"os"
-
 	"github.com/codegangsta/cli"
+	"github.com/recruit-mp/kangol/internal/operation"
 )
 
-func main() {
+func NewApp(version string) *cli.App {
 	finished := make(chan bool)
-	go loading(finished)
+	go operation.Loading(finished)
 
 	app := cli.NewApp()
 	app.Name = "kangol"
-	app.Version = "0.2.8"
+	app.Version = version
 	app.Usage = "ECS deployment tool"
 	app.Flags = []cli.Flag{
 		cli.StringFlag{
@@ -49,7 +48,7 @@ func main() {
 		if c.Bool("loading") == false {
 			finished <- true
 		}
-		deploy(
+		operation.Deploy(
 			c.String("conf"),
 			c.String("tag"),
 			c.Bool("debug"),
@@ -89,11 +88,9 @@ func main() {
 				},
 			},
 			Action: func(c *cli.Context) {
-				runTask(c.String("conf"), c.String("tag"), c.String("command"), c.Int64("cpu"), c.Int64("memory"))
+				operation.RunTask(c.String("conf"), c.String("tag"), c.String("command"), c.Int64("cpu"), c.Int64("memory"))
 			},
 		},
 	}
-
-	app.Run(os.Args)
-
+	return app
 }

deploy.go → internal/operation/deploy.go

@@ -1,16 +1,16 @@
-package main
+package operation
 
 import (
 	"fmt"
 	"strings"
 	"time"
 
 	log "github.com/Sirupsen/logrus"
-	"github.com/recruit-mp/kangol/awsecs"
-	"github.com/recruit-mp/kangol/task"
+	"github.com/recruit-mp/kangol/internal/awsecs"
+	"github.com/recruit-mp/kangol/internal/task"
 )
 
-func deploy(conf, tag string, debug bool, skipPolling bool, pollingTime int64) {
+func Deploy(conf, tag string, debug bool, skipPolling bool, pollingTime int64) {
 
 	deployment, taskDefinition, err := task.ReadConfig(conf, appendTags(tag))
 
@@ -93,7 +93,7 @@ func appendTags(tag string) map[string]string {
 	return tags
 }
 
-func loading(finished chan bool) {
+func Loading(finished chan bool) {
 loop:
 	for {
 		select {

run_task.go → internal/operation/run_task.go

@@ -1,17 +1,17 @@
-package main
+package operation
 
 import (
 	"fmt"
 	"strings"
 
 	log "github.com/Sirupsen/logrus"
 	"github.com/aws/aws-sdk-go/aws"
-	"github.com/recruit-mp/kangol/awscloudwatchlogs"
-	"github.com/recruit-mp/kangol/awsecs"
-	"github.com/recruit-mp/kangol/task"
+	"github.com/recruit-mp/kangol/internal/awscloudwatchlogs"
+	"github.com/recruit-mp/kangol/internal/awsecs"
+	"github.com/recruit-mp/kangol/internal/task"
 )
 
-func runTask(conf, tag string, command string, cpu int64, memory int64) {
+func RunTask(conf, tag string, command string, cpu int64, memory int64) {
 
 	deployment, taskDefinition, err := task.ReadConfig(conf, appendTags(tag))