Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove an duplicate list of parser directives #21314

Closed

Conversation

t-dub
Copy link

@t-dub t-dub commented Nov 1, 2024

Description

The Dockerfile reference has an extra -- incomplete and unlinked -- set of parser directives included in it. I'm assuming it was just accidentally left over from a previous revision

image

Further up the page is the "real" section:

image

Dockerfile reference has an extra unlinked list of parser directives;
remove it.
@github-actions github-actions bot added area/build Relates to Dockerfiles or docker build command dependencies Vendoring, packaging updates labels Nov 1, 2024
Copy link

netlify bot commented Nov 1, 2024

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit 8ad83ca
🔍 Latest deploy log https://app.netlify.com/sites/docsdocker/deploys/6725376c569626000878469e
😎 Deploy Preview https://deploy-preview-21314--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.

Copy link
Collaborator

@dvdksn dvdksn left a comment

Choose a reason for hiding this comment

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

Thank you! This file lives in the moby/buildkit repository. We'll need to update it there, would you mind raising a PR on that repository? The fix will be vendored here when it's merged upstream. Thank you!

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 dependencies Vendoring, packaging updates
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants