fix: docker stub and add a test for this (#2355)

* fix: docker stub * test if you can build run without docker --------- Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
nektos · Jul 10, 2024 · 3c7eda7 · 3c7eda7
1 parent a1a96da
commit 3c7eda7
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/.github/workflows/checks.yml b/.github/workflows/checks.yml
@@ -60,6 +60,8 @@ jobs:
           upload-logs-name: logs-linux
       - name: Run act from cli
         run: go run main.go -P ubuntu-latest=node:16-buster-slim -C ./pkg/runner/testdata/ -W ./basic/push.yml
+      - name: Run act from cli without docker support
+        run: go run -tags WITHOUT_DOCKER main.go -P ubuntu-latest=-self-hosted -C ./pkg/runner/testdata/ -W ./local-action-js/push.yml
       - name: Upload Codecov report
         uses: codecov/codecov-action@v4
         with:

diff --git a/pkg/container/docker_stub.go b/pkg/container/docker_stub.go
@@ -6,7 +6,7 @@ import (
 	"context"
 	"runtime"
 
-	"github.com/docker/docker/api/types"
+	"github.com/docker/docker/api/types/system"
 	"github.com/nektos/act/pkg/common"
 	"github.com/pkg/errors"
 )
@@ -46,8 +46,8 @@ func RunnerArch(ctx context.Context) string {
 	return runtime.GOOS
 }
 
-func GetHostInfo(ctx context.Context) (info types.Info, err error) {
-	return types.Info{}, nil
+func GetHostInfo(ctx context.Context) (info system.Info, err error) {
+	return system.Info{}, nil
 }
 
 func NewDockerVolumeRemoveExecutor(volume string, force bool) common.Executor {