Skip to content
This repository has been archived by the owner on Nov 27, 2023. It is now read-only.

Add quietPull option to up #1390

Merged
merged 1 commit into from
Mar 11, 2021
Merged

Conversation

ulyssessouza
Copy link
Contributor

@ulyssessouza ulyssessouza commented Mar 4, 2021

What I did
Add quietPull option to up

Resolves #1368
Depends on docker/buildx#558

@ulyssessouza ulyssessouza requested review from gtardif and ndeloof March 4, 2021 05:48
@github-actions github-actions bot added api api cli cli local Local context (moby) labels Mar 4, 2021
local/compose/build.go Outdated Show resolved Hide resolved
@ulyssessouza ulyssessouza marked this pull request as ready for review March 4, 2021 05:52
Copy link
Contributor

@gtardif gtardif left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just add more explicit param to service.build rather than mixing behaviour between tty display and quiet pull option, otherwise LGTM

@ulyssessouza ulyssessouza marked this pull request as draft March 5, 2021 01:42
@ulyssessouza ulyssessouza force-pushed the quiet-pull branch 2 times, most recently from 57092f5 to d14c59c Compare March 5, 2021 03:19
@gtardif
Copy link
Contributor

gtardif commented Mar 8, 2021

This is waiting on docker/buildx#558

Signed-off-by: Ulysses Souza <[email protected]>
@ulyssessouza ulyssessouza requested a review from gtardif March 10, 2021 14:46
@ulyssessouza ulyssessouza marked this pull request as ready for review March 10, 2021 14:46
@gtardif gtardif merged commit 9301c29 into docker-archive:main Mar 11, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
api api cli cli local Local context (moby)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support compose up --quiet-pull
3 participants