Guest24897
/
DockerCLI
mirror of https://github.com/docker/cli.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #28153 from YuPengZTE/devCtx 

context.Context should be the first parameter of a function
This commit is contained in:
Vincent Demeester 2016-11-08 12:40:32 +01:00 committed by GitHub
parent 6edfb7558f 7113bbf2c6
commit 544b21fb65
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
command/container/start.go
View File

@ -155,13 +155,13 @@ func runStart(dockerCli *command.DockerCli, opts *startOptions) error {
} else { } else {
// We're not going to attach to anything. // We're not going to attach to anything.
// Start as many containers as we want. // Start as many containers as we want.
return startContainersWithoutAttachments(dockerCli, ctx, opts.containers) return startContainersWithoutAttachments(ctx, dockerCli, opts.containers)
} }
return nil return nil
} }
func startContainersWithoutAttachments(dockerCli *command.DockerCli, ctx context.Context, containers []string) error { func startContainersWithoutAttachments(ctx context.Context, dockerCli *command.DockerCli, containers []string) error {
var failedContainers []string var failedContainers []string
for _, container := range containers { for _, container := range containers {
if err := dockerCli.Client().ContainerStart(ctx, container, types.ContainerStartOptions{}); err != nil { if err := dockerCli.Client().ContainerStart(ctx, container, types.ContainerStartOptions{}); err != nil {