Skip to content

Commit bb40664

Browse files
author
Mrunal Patel
authored
Merge pull request #1305 from giuseppe/kill-max-2-args
kill: requires max 2 arguments
2 parents 9073486 + 1d64477 commit bb40664

File tree

2 files changed

+8
-0
lines changed

2 files changed

+8
-0
lines changed

kill.go

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -72,6 +72,9 @@ signal to the init process of the "ubuntu01" container:
7272
if err := checkArgs(context, 1, minArgs); err != nil {
7373
return err
7474
}
75+
if err := checkArgs(context, 2, maxArgs); err != nil {
76+
return err
77+
}
7578
container, err := getContainer(context)
7679
if err != nil {
7780
return err

utils.go

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@ import (
1313
const (
1414
exactArgs = iota
1515
minArgs
16+
maxArgs
1617
)
1718

1819
func checkArgs(context *cli.Context, expected, checkType int) error {
@@ -27,6 +28,10 @@ func checkArgs(context *cli.Context, expected, checkType int) error {
2728
if context.NArg() < expected {
2829
err = fmt.Errorf("%s: %q requires a minimum of %d argument(s)", os.Args[0], cmdName, expected)
2930
}
31+
case maxArgs:
32+
if context.NArg() > expected {
33+
err = fmt.Errorf("%s: %q requires a maximum of %d argument(s)", os.Args[0], cmdName, expected)
34+
}
3035
}
3136

3237
if err != nil {

0 commit comments

Comments
 (0)