Conversation
Test Results792 tests 792 ✅ 8m 59s ⏱️ Results for commit 94c6a8e. ♻️ This comment has been updated with latest results. |
| plan: Annotated[Optional[bool], typer.Option(help="Print the bake plan and exit.")] = False, | ||
| load: Annotated[Optional[bool], typer.Option(help="Load the image to Docker after building.")] = True, | ||
| push: Annotated[Optional[bool], typer.Option(help="Push the image to the registry after building.")] = False, | ||
| platform: Annotated[ |
There was a problem hiding this comment.
I'm confused about the singular vs plural here.
Does it have the ability to specify the --platform flag multiple times?
There was a problem hiding this comment.
Yes, but you have to do in the style of --platform linux/amd64 --platform linux/arm64. I guess I thought it might be confusing to do --platforms since that would imply in my mind that you should do --platforms linux/amd64,linux/arm64 or something like that. It would be possible to do it that way of course. I don't know, do you have an opinion on which is easier?
There was a problem hiding this comment.
Perfect! I agree, specifying it each time is more clear.
Closes #262
platformsconfiguration option toImageVersionOSto specify target platforms to build for an image.--platformmulti-entry option forbakery buildto override the targeted build platforms at runtime.linux/amd64for all images if no configuration or option is given.