Merge pull request #17 from sambaiz/feature/pre_command

preCommands option

README.md

@@ -83,6 +83,9 @@ targets:
   master:
     contexts:
       env: prd
+preCommands:
+  # Optional. Run before command.
+  - npm run build
 deployUsers:
   # Optional. If specified, only these users are allowed to deploy.
   # If not, all users are allowed to deploy.

functions/operation/command/command.go

@@ -8,6 +8,7 @@ import (
 	"github.com/sambaiz/cdkbot/functions/operation/constant"
 	"github.com/sambaiz/cdkbot/functions/operation/git"
 	"github.com/sambaiz/cdkbot/functions/operation/platform"
+	"os/exec"
 	"regexp"
 	"strings"
 )
@@ -126,6 +127,11 @@ func (r *Runner) setup(ctx context.Context, cloneHead bool) (string, *config.Con
 	if err := r.cdk.Setup(cdkPath); err != nil {
 		return "", nil, nil, nil, err
 	}
+
+	for _, preCommand := range cfg.PreCommands {
+		command := strings.Split(preCommand, " ")
+		exec.Command(command[0], command[1:]...)
+	}
 	return cdkPath, cfg, &target, pr, nil
 }
 

functions/operation/config/config.go

@@ -18,6 +18,7 @@ type Reader struct{}
 type Config struct {
 	CDKRoot     string            `yaml:"cdkRoot"`
 	Targets     map[string]Target `yaml:"targets"`
+	PreCommands []string          `yaml:"preCommands"`
 	DeployUsers []string          `yaml:"deployUsers"`
 }
 

functions/operation/config/config_test.go

@@ -30,6 +30,7 @@ func TestReaderRead(t *testing.T) {
 						},
 					},
 				},
+				PreCommands: []string{"npm run build"},
 				DeployUsers: []string{"sambaiz"},
 			},
 		},

functions/operation/config/test_config/cdkbot.yml

@@ -6,5 +6,7 @@ targets:
   master:
     contexts:
       env: prd
+preCommands:
+  - npm run build
 deployUsers:
   - sambaiz