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
Add missing options to bash completion for docker build
#17107
Conversation
docker build
docker build
Signed-off-by: Harald Albers <github@albersweb.de>
Signed-off-by: Harald Albers <github@albersweb.de>
2651891
to
3a713d9
Compare
Added this to 1.9.0 milestone (as the command is in there). |
LGTM |
--cgroup-parent | ||
--cpuset-cpus | ||
--cpuset-mems | ||
--cpu-shares -c |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
arg this is on build... we need to remove it from build too :(
EDIT see #17117
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Handling it here #17119 😉
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@vdemeester Thanks for taking care of -c
.
LGTM |
Add missing options to bash completion for `docker build`
Ref: #15182
This adds the new
--build-arg
and the missing--disable-content-trust
to bash completion fordocker build
.Hint: the completion of
--build-arg
is the same as fordocker run -e
.As the option lists got too long, I refactored it to use the idioms used in completion for
docker run
.Thanks @sdurrheimer for the ping.
As this PR is not trivial, ping @jfrazelle, @tianon for review.