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.

Sign up for GitHub

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

Jump to bottom

Hovers do not resolve properly for uninitialized ARG variables that share the same with one at the top of a Dockerfile #34

Closed
rcjsuen opened this issue May 13, 2018 · 3 comments
Closed

Hovers do not resolve properly for uninitialized ARG variables that share the same with one at the top of a Dockerfile #34

rcjsuen opened this issue May 13, 2018 · 3 comments
Assignees
@rcjsuen
Labels
bug Something isn't working

Comments

@rcjsuen
Copy link
Owner

rcjsuen commented May 13, 2018

Hovering over $ver on the fourth line does not give latest.

ARG ver=latest
FROM busybox:$ver
ARG ver
RUN echo $ver
$ docker build .
Sending build context to Docker daemon  1.693MB
Step 1/4 : ARG ver=latest
Step 2/4 : FROM busybox:$ver
latest: Pulling from library/busybox
f70adabe43c0: Pull complete
Digest: sha256:58ac43b2cc92c687a32c8be6278e50a063579655fe3090125dcb2af0ff9e1a64
Status: Downloaded newer image for busybox:latest
 ---> 8ac48589692a
Step 3/4 : ARG ver
 ---> Running in af5ced8a8d4c
Removing intermediate container af5ced8a8d4c
 ---> 0a3042e16719
Step 4/4 : RUN echo $ver
 ---> Running in a74c14da9ae4
latest
Removing intermediate container a74c14da9ae4
 ---> 370fcb2b140c
Successfully built 370fcb2b140c
@rcjsuen rcjsuen added the bug Something isn't working label May 13, 2018
@rcjsuen rcjsuen self-assigned this May 13, 2018
@rcjsuen
Copy link
Owner Author

rcjsuen commented May 18, 2018

rcjsuen/dockerfile-ast#26 has been fixed. Will need to adopt that fix and see if anything else needs to be done on this side.

@rcjsuen
Copy link
Owner Author

rcjsuen commented May 29, 2018

Blocked by rcjsuen/dockerfile-ast#42 now.

@rcjsuen
Copy link
Owner Author

rcjsuen commented Jun 16, 2018

Now blocked by rcjsuen/dockerfile-ast#44. :(

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rcjsuen rcjsuen

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@rcjsuen