Skip to content

Correct no-cache attribute to be a boolean#3473

Merged
crazy-max merged 1 commit intodocker:masterfrom
rcjsuen:no-cache-docs-fix
Oct 17, 2025
Merged

Correct no-cache attribute to be a boolean#3473
crazy-max merged 1 commit intodocker:masterfrom
rcjsuen:no-cache-docs-fix

Conversation

@rcjsuen
Copy link
Contributor

@rcjsuen rcjsuen commented Oct 16, 2025

The documentation is incorrect because no-cache needs to be a boolean.

$ docker buildx version
github.com/docker/buildx v0.29.1-desktop.1 28f6246ff24e2c05095e8741e48c48dcb2d3b4bc
target "default" {
  no-cache = 1
}
$ docker buildx bake --print default
[+] Building 0.0s (1/1) FINISHED
 => [internal] load local bake definitions                                                                                                                                  0.0s
 => => reading docker-bake.hcl 36B / 36B                                                                                                                                    0.0s
docker-bake.hcl:2
--------------------
   1 |     target "default" {
   2 | >>>   no-cache = 1
   3 |     }
   4 |
--------------------
ERROR: docker-bake.hcl:2,14-15: Unsuitable value type; Unsuitable value: bool required, but have number

The documentation is incorrect because no-cache needs to be a boolean.

Signed-off-by: Remy Suen <remy.suen@docker.com>
Copy link
Member

@crazy-max crazy-max left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@crazy-max crazy-max merged commit f8fb2ab into docker:master Oct 17, 2025
41 checks passed
@rcjsuen rcjsuen deleted the no-cache-docs-fix branch October 17, 2025 09:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants