Merge pull request #1974 from appleboy/matrix

fix: missing matrix env.
2017-03-21 23:22:24 +08:00 · 2017-03-21 23:22:24 +08:00 · e5e965b797
commit e5e965b797
parent f0cbe62d81 316a6ea454
1 changed files with 8 additions and 2 deletions
--- a/server/hook.go
+++ b/server/hook.go
@ -422,6 +422,10 @@ func (b *builder) Build() ([]*buildItem, error) {
 			environ[k] = v
 		}

+		for k, v := range axis {
+			environ[k] = v
+		}
+
 		secrets := map[string]string{}
 		for _, sec := range b.Secs {
 			if !sec.MatchEvent(b.Curr.Event) {
@ -439,9 +443,11 @@ func (b *builder) Build() ([]*buildItem, error) {
 		}

 		y := b.Yaml
-		if s, err := envsubst.Eval(y, sub); err != nil {
-			y = s
+		s, err := envsubst.Eval(y, sub)
+		if err != nil {
+			return nil, err
 		}
+		y = s

 		parsed, err := yaml.ParseString(y)
 		if err != nil {