fix 99: support string and map variants of container

act/runner/run_context.go

@@ -53,8 +53,9 @@ func (rc *RunContext) startJobContainer() common.Executor {
 	job := rc.Run.Job()
 
 	var image string
-	if job.Container != nil {
-		image = job.Container.Image
+	c := job.Container()
+	if c != nil {
+		image = c.Image
 	} else {
 		platformName := rc.ExprEval.Interpolate(job.RunsOn)
 		image = rc.Config.Platforms[strings.ToLower(platformName)]

act/runner/step_context.go

@@ -87,8 +87,9 @@ func (sc *StepContext) setupEnv() common.Executor {
 	step := sc.Step
 	return func(ctx context.Context) error {
 		var env map[string]string
-		if job.Container != nil {
-			env = mergeMaps(rc.GetEnv(), job.Container.Env, step.GetEnv())
+		c := job.Container()
+		if c != nil {
+			env = mergeMaps(rc.GetEnv(), c.Env, step.GetEnv())
 		} else {
 			env = mergeMaps(rc.GetEnv(), step.GetEnv())
 		}

act/runner/testdata/job-container/push.yml

@@ -10,3 +10,11 @@ jobs:
         TEST_ENV: test-value
     steps:
       - run: echo ${TEST_ENV} | grep test-value
+
+  test2:
+    runs-on: ubuntu-latest
+    container: node:10.16-jessie
+    steps:
+      - run: echo ${TEST_ENV} | grep test-value
+        env:
+          TEST_ENV: test-value