Skip to content

docs: add missing documentation for --pull flag - #3449

Merged
thaJeztah merged 1 commit into
docker:masterfrom
thaJeztah:docs_add_missing_pull_policy
Mar 1, 2022
Merged

docs: add missing documentation for --pull flag#3449
thaJeztah merged 1 commit into
docker:masterfrom
thaJeztah:docs_add_missing_pull_policy

Conversation

@thaJeztah

Copy link
Copy Markdown
Member

addresses docker/docs#14135

These options were added in 22cd418 (#1498), but did not update the documentation. That PR was already included in docker 20.10, so adding the cherry-pick label as well.

I noticed this when looking at #3377 (I started this PR with that commit)

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

Merging #3449 (9f88f3c) into master (6c6203f) will not change coverage.
The diff coverage is n/a.

@@ Coverage Diff @@## master #3449 +/- ##
=======================================
Coverage 58.97% 58.97% =======================================
Files 284 284 Lines 23834 23834 =======================================
Hits 14057 14057 Misses 8918 8918 Partials 859 859 

These options were added in 22cd418,
but did not update the documentation.
Signed-off-by: Chee Hau Lim <cheehau.lim@mobimeo.com>
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah
thaJeztahforce-pushed the docs_add_missing_pull_policy branch from 9f88f3c to 7eb61e2CompareMarch 1, 2022 10:40
@thaJeztah

Copy link
Copy Markdown
MemberAuthor

@ndeloof@rumpl ptal 🤗

@ndeloofndeloof left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@thaJeztah

Copy link
Copy Markdown
MemberAuthor

Ah, nice, yes. Forgot for a bit that it also defines it (but I recall we had a discussion on making them match as much as possible)

Perhaps #3377 is also a candidate to add to docker compose run btw

Sign up for freeto 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.

4 participants

@thaJeztah@codecov-commenter@ndeloof@ch33hau