Skip to content

vendor: moby/client v0.4.0, moby/api v1.54.1#3779

Open
thaJeztah wants to merge 1 commit intodocker:masterfrom
thaJeztah:bump_moby
Open

vendor: moby/client v0.4.0, moby/api v1.54.1#3779
thaJeztah wants to merge 1 commit intodocker:masterfrom
thaJeztah:bump_moby

Conversation

@thaJeztah
Copy link
Copy Markdown
Member

No description provided.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
}
defer resp.Close()
return jsonmessage.DisplayJSONMessagesStream(resp, io.Discard, 0, false, nil)
return jsonmessage.DisplayStream(resp, io.Discard)
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This one just delegates TTY detection (in this case "no TTY"), so that it's not needed to pass it in; see

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Ah yes sounds better than #3777

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants