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

No warning for error "ENV must have two arguments" #48

Closed
lucasdf opened this issue Dec 29, 2017 · 1 comment
Closed

No warning for error "ENV must have two arguments" #48

lucasdf opened this issue Dec 29, 2017 · 1 comment

Comments

@lucasdf
Copy link

lucasdf commented Dec 29, 2017

When declaring an ENV variable its value must be declared too.

When running the linter for the following content of a Dockerfile on earlier versions of dockerlint I received the error ERROR: ENV invalid format ENV_VARIABLE. However, the same error is not displayed anymore since the release 0.3.6

FROM  someimage
ENV  ENV_VARIABLE

ENV requires to be followed by both a key and a value. When trying to build a Dockerfile containing the content above, Docker displays Error response from daemon: ENV must have two arguments.

So, it seems that dockerlint should go back to the earlier behavior and display an error message in such situation.

Related to #46

@lucasdf
Copy link
Author

lucasdf commented Jan 2, 2018

awesome! thanks!

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

1 participant