Skip to content
This repository has been archived by the owner on Jun 16, 2021. It is now read-only.

Add support for --no-build #136

Merged
merged 1 commit into from
Jan 27, 2016
Merged

Conversation

vdemeester
Copy link
Collaborator

This adds support for the --no-build flag for create and up commands.

Also contains a small refactoring on how libcompose decides to build or
not the image (it aligns with how docker-compose does). 🐠

  • What to do about the FIXMEs
  • Add some integration tests

🐸

Signed-off-by: Vincent Demeester vincent@sbr.pm

@runcom
Copy link

runcom commented Jan 23, 2016

I just have a small nit regarding the imageName param, I guess compose is fine with this but I'd rather keep using tag or just image. Is compose making difference between repo and repo:tag?

@vdemeester
Copy link
Collaborator Author

@runcom just image looks fine to me. The image you specify in the yaml is repo or repo:tag (if no tag specified, we use the default one, latest).

@runcom
Copy link

runcom commented Jan 23, 2016

@vdemeester right cool

@vdemeester vdemeester force-pushed the no-build-support branch 3 times, most recently from dd8f5ae to ac946a9 Compare January 23, 2016 13:04
This adds support for the --no-build flag for create and up commands.

Also contains a small refactoring on how libcompose decides to build or
not the image (it aligns with how docker-compose does).

Signed-off-by: Vincent Demeester <vincent@sbr.pm>
@vdemeester
Copy link
Collaborator Author

/ping @ibuildthecloud @dnephin

vdemeester added a commit that referenced this pull request Jan 27, 2016
@vdemeester vdemeester merged commit f339ae6 into docker:master Jan 27, 2016
@vdemeester vdemeester deleted the no-build-support branch January 27, 2016 21:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants