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

fromlatest.io still errors on having ARG before FROM #135

Closed
hansbogert opened this issue Oct 11, 2019 · 7 comments
Closed

fromlatest.io still errors on having ARG before FROM #135

hansbogert opened this issue Oct 11, 2019 · 7 comments

Comments

@hansbogert
Copy link

Currently, the linter gives warning of 'First Command Must Be FROM (Possible Bug)'. However, I think there is at least one exception:

In order to have access to build arguments in multiple stages, said build arguments should be declared in the outer scope of the Dockerfile, before any FROM directives

@deiga
Copy link
Contributor

deiga commented Nov 13, 2019

@hansbogert This should already be working, see #105

@hansbogert
Copy link
Author

hansbogert commented Nov 13, 2019

strange, the online variant still shows this behaviour

@deiga
Copy link
Contributor

deiga commented Nov 13, 2019

My hunch would be, that the online variant is not using the newest version. It doesn't have the version listed anywhere, so hard to check

@deiga
Copy link
Contributor

deiga commented Nov 13, 2019

Could you reword the title to reflect that the issue is with the online variant? :)

@hansbogert hansbogert changed the title It should be possible to have ARG before the first FROM fromlatest.io still errors on having ARG before FROM Nov 15, 2019
@asears
Copy link
Contributor

asears commented Jul 31, 2020

This still seems to be happening in https://www.fromlatest.io/#/

ARG is valid syntax before FROM as of moby/moby#31352

@SimonHeimberg
Copy link

SimonHeimberg commented Sep 30, 2020

The online version seems to be updated. At least the following examples works. Can we close this issue?

ARG xxx

FROM ubuntu:bionic

@hansbogert
Copy link
Author

@SimonHeimberg seems to work correctly indeed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants