Add support for comma separated IntSliceFlag and Int64SliceFlag

flag_int64_slice.go

@@ -33,6 +33,18 @@ func (i *Int64Slice) Set(value string) error {
 		return nil
 	}
 
+	if strings.Contains(value, ",") {
+		values := strings.Split(value, ",")
+		for _, v := range values {
+			tmp, err := strconv.ParseInt(v, 0, 64)
+			if err != nil {
+				return err
+			}
+			i.slice = append(i.slice, tmp)
+		}
+		return nil
+	}
+
 	tmp, err := strconv.ParseInt(value, 0, 64)
 	if err != nil {
 		return err

flag_int_slice.go

@@ -44,6 +44,18 @@ func (i *IntSlice) Set(value string) error {
 		return nil
 	}
 
+	if strings.Contains(value, ",") {
+		values := strings.Split(value, ",")
+		for _, v := range values {
+			tmp, err := strconv.ParseInt(v, 0, 64)
+			if err != nil {
+				return err
+			}
+			i.slice = append(i.slice, int(tmp))
+		}
+		return nil
+	}
+
 	tmp, err := strconv.ParseInt(value, 0, 64)
 	if err != nil {
 		return err

flag_test.go

@@ -1196,6 +1196,28 @@ func TestParseMultiIntSlice(t *testing.T) {
 	}).Run([]string{"run", "-s", "10", "-s", "20"})
 }
 
+func TestParseMultiIntSliceCommaSeperated(t *testing.T) {
+	//Need to check for error since the parsing fails before it gets into the Action
+	//Then this test will fail without being noticed
+	err := (&App{
+		Flags: []Flag{
+			&IntSliceFlag{Name: "serve", Aliases: []string{"s"}, Value: NewIntSlice()},
+		},
+		Action: func(ctx *Context) error {
+			if !reflect.DeepEqual(ctx.IntSlice("serve"), []int{10, 20}) {
+				t.Errorf("main name not set")
+			}
+			if !reflect.DeepEqual(ctx.IntSlice("s"), []int{10, 20}) {
+				t.Errorf("short name not set")
+			}
+			return nil
+		},
+	}).Run([]string{"run", "-s", "10,20"})
+	if err != nil {
+		t.Errorf("failed to parse flags : %s", err)
+	}
+}
+
 func TestParseMultiIntSliceWithDefaults(t *testing.T) {
 	_ = (&App{
 		Flags: []Flag{
@@ -1310,6 +1332,28 @@ func TestParseMultiInt64Slice(t *testing.T) {
 	}).Run([]string{"run", "-s", "10", "-s", "17179869184"})
 }
 
+func TestParseMultiInt64SliceCommaSeperated(t *testing.T) {
+	//Need to check for error since the parsing fails before it gets into the Action
+	//Then this test will fail without being noticed
+	err := (&App{
+		Flags: []Flag{
+			&Int64SliceFlag{Name: "serve", Aliases: []string{"s"}, Value: NewInt64Slice()},
+		},
+		Action: func(ctx *Context) error {
+			if !reflect.DeepEqual(ctx.Int64Slice("serve"), []int64{10, 17179869184}) {
+				t.Errorf("main name not set")
+			}
+			if !reflect.DeepEqual(ctx.Int64Slice("s"), []int64{10, 17179869184}) {
+				t.Errorf("short name not set")
+			}
+			return nil
+		},
+	}).Run([]string{"run", "-s", "10,17179869184"})
+	if err != nil {
+		t.Errorf("failed to parse flags : %s", err)
+	}
+}
+
 func TestParseMultiInt64SliceFromEnv(t *testing.T) {
 	defer resetEnv(os.Environ())
 	os.Clearenv()