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

Make service>build>dockerfile a simple filename #8779

Merged
merged 2 commits into from
Oct 20, 2021

Conversation

ulyssessouza
Copy link
Collaborator

@ulyssessouza ulyssessouza commented Oct 11, 2021

Resolves #8760

What I did

  • It makes it keep a simple filename instead of an absolute path

Related issue

(not mandatory) A picture of a cute animal, if possible in relation with what you did

@ulyssessouza ulyssessouza requested a review from ndeloof October 11, 2021 21:36
@ulyssessouza
Copy link
Collaborator Author

Should still a draft until compose-spec/compose-go#195 gets merged

@ulyssessouza ulyssessouza force-pushed the build-dockerfile-relativepath branch from f3999f9 to 3f9a3b2 Compare October 11, 2021 21:50
cmd/compose/convert.go Show resolved Hide resolved
@ulyssessouza ulyssessouza force-pushed the build-dockerfile-relativepath branch from 3f9a3b2 to 92720d7 Compare October 12, 2021 07:22
@ulyssessouza ulyssessouza marked this pull request as ready for review October 14, 2021 04:58
go.mod Outdated Show resolved Hide resolved
@ulyssessouza ulyssessouza force-pushed the build-dockerfile-relativepath branch 4 times, most recently from e7ff923 to 1047c5b Compare October 18, 2021 15:45
- It makes it keep a simple filename instead of an absolute path

Signed-off-by: Ulysses Souza <[email protected]>
@ulyssessouza ulyssessouza force-pushed the build-dockerfile-relativepath branch from 1047c5b to 9437976 Compare October 20, 2021 08:36
@ulyssessouza ulyssessouza merged commit c3a5eb2 into docker:v2 Oct 20, 2021
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

Successfully merging this pull request may close these issues.

Output of "docker compose config" does not contain full path to context
4 participants