Merge pull request #6 from erizocosmico/patch-1

mcuadros · web-flow · commit 5781925c1959 · 2024-10-21T20:36:18.000+02:00
docker: register correct builtin function for Image.tags()
diff --git a/starlark/module/docker/image.go b/starlark/module/docker/image.go
@@ -144,7 +144,7 @@ func (i *image) Attr(name string) (starlark.Value, error) {
 		name := i.ref.DockerReference()
 		return starlark.String(reference.Path(name)), nil
 	case "tags":
-		return starlark.NewBuiltin("tags", i.builtinVersionFunc), nil
+		return starlark.NewBuiltin("tags", i.builtinTagsFunc), nil
 	case "version":
 		return starlark.NewBuiltin("version", i.builtinVersionFunc), nil
 	}

Original file line number	Diff line number	Diff line change
`@@ -144,7 +144,7 @@ func (i *image) Attr(name string) (starlark.Value, error) {`
`144`	`144`	`name := i.ref.DockerReference()`
`145`	`145`	`return starlark.String(reference.Path(name)), nil`
`146`	`146`	`case "tags":`
`147`		`- return starlark.NewBuiltin("tags", i.builtinVersionFunc), nil`
	`147`	`+ return starlark.NewBuiltin("tags", i.builtinTagsFunc), nil`
`148`	`148`	`case "version":`
`149`	`149`	`return starlark.NewBuiltin("version", i.builtinVersionFunc), nil`
`150`	`150`	`}`