Skip to content

Commit 6de2cdd

Browse files
committed
cli/command/task: inline vars and use struct literals
Signed-off-by: Sebastiaan van Stijn <[email protected]>
1 parent e308036 commit 6de2cdd

File tree

1 file changed

+22
-16
lines changed

1 file changed

+22
-16
lines changed

cli/command/task/formatter.go

Lines changed: 22 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -55,27 +55,33 @@ func FormatWrite(fmtCtx formatter.Context, tasks []swarm.Task, names map[string]
5555

5656
// formatWrite writes the context.
5757
func formatWrite(fmtCtx formatter.Context, tasks []swarm.Task, names map[string]string, nodes map[string]string) error {
58-
render := func(format func(subContext formatter.SubContext) error) error {
58+
taskCtx := &taskContext{
59+
HeaderContext: formatter.HeaderContext{
60+
Header: formatter.SubHeaderContext{
61+
"ID": taskIDHeader,
62+
"Name": formatter.NameHeader,
63+
"Image": formatter.ImageHeader,
64+
"Node": nodeHeader,
65+
"DesiredState": desiredStateHeader,
66+
"CurrentState": currentStateHeader,
67+
"Error": formatter.ErrorHeader,
68+
"Ports": formatter.PortsHeader,
69+
},
70+
},
71+
}
72+
return fmtCtx.Write(taskCtx, func(format func(subContext formatter.SubContext) error) error {
5973
for _, task := range tasks {
60-
taskCtx := &taskContext{trunc: fmtCtx.Trunc, task: task, name: names[task.ID], node: nodes[task.ID]}
61-
if err := format(taskCtx); err != nil {
74+
if err := format(&taskContext{
75+
trunc: fmtCtx.Trunc,
76+
task: task,
77+
name: names[task.ID],
78+
node: nodes[task.ID],
79+
}); err != nil {
6280
return err
6381
}
6482
}
6583
return nil
66-
}
67-
taskCtx := taskContext{}
68-
taskCtx.Header = formatter.SubHeaderContext{
69-
"ID": taskIDHeader,
70-
"Name": formatter.NameHeader,
71-
"Image": formatter.ImageHeader,
72-
"Node": nodeHeader,
73-
"DesiredState": desiredStateHeader,
74-
"CurrentState": currentStateHeader,
75-
"Error": formatter.ErrorHeader,
76-
"Ports": formatter.PortsHeader,
77-
}
78-
return fmtCtx.Write(&taskCtx, render)
84+
})
7985
}
8086

8187
type taskContext struct {

0 commit comments

Comments
 (0)