Skip to content

Conversation

@dvdksn
Copy link
Contributor

@dvdksn dvdksn commented Apr 12, 2024

Description

Add instruction for testing the experimental buildkit.exe support on Windows, with Buildx

Related issues or tickets

Carries moby/buildkit#4835

Reviews

@dvdksn dvdksn requested a review from crazy-max as a code owner April 12, 2024 08:34
@github-actions github-actions bot added the area/build Relates to Dockerfiles or docker build command label Apr 12, 2024
@netlify
Copy link

netlify bot commented Apr 12, 2024

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit 2b13711
🔍 Latest deploy log https://app.netlify.com/sites/docsdocker/deploys/661ceb008be9640008eb15af
😎 Deploy Preview https://deploy-preview-19792--docsdocker.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@dvdksn dvdksn self-assigned this Apr 12, 2024
crazy-max
crazy-max previously approved these changes Apr 12, 2024
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.

LGTM

@dvdksn dvdksn requested a review from a team April 12, 2024 08:58
@colinhemmings
Copy link

colinhemmings commented Apr 12, 2024

@dvdksn There are a few instances where the command and the output are shown in the same code block. Should we separate them to make it easier for users to just copy the relevant commands?

Copy link
Contributor

@profnandaa profnandaa left a comment

Choose a reason for hiding this comment

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

The rest LGTM, thanks!

Signed-off-by: David Karlsson <35727626+dvdksn@users.noreply.github.com>
@dvdksn dvdksn force-pushed the buildkit-windows-exp branch from b6f1bef to 2b13711 Compare April 15, 2024 08:53
Copy link
Contributor

@profnandaa profnandaa left a comment

Choose a reason for hiding this comment

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

LGTM.

@dvdksn dvdksn merged commit 37f10b3 into docker:main Apr 16, 2024
@dvdksn dvdksn deleted the buildkit-windows-exp branch April 16, 2024 06:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/build Relates to Dockerfiles or docker build command status/review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants