cmd/flags/csvflag.go

@@ -19,20 +19,22 @@ type CsvFlag struct {
 	AvailableFields []string
 }
 
-func NewCsvFlag(name, usage string, aliases, availableFields, defaultFields []string) *CsvFlag {
+// NewCsvFlag creates a CsvFlag, while setting its usage string and default values
+func NewCsvFlag(name, usage string, aliases, availableValues, defaults []string) *CsvFlag {
 	return &CsvFlag{
-		AvailableFields: availableFields,
+		AvailableFields: availableValues,
 		StringFlag: cli.StringFlag{
 			Name:    name,
 			Aliases: aliases,
-			Value:   strings.Join(defaultFields, ","),
+			Value:   strings.Join(defaults, ","),
 			Usage: fmt.Sprintf(`Comma-separated list of %s. Available values:
 			%s
-		`, usage, strings.Join(availableFields, ",")),
+		`, usage, strings.Join(availableValues, ",")),
 		},
 	}
 }
 
+// GetValues returns the value of the flag, parsed as a commaseparated list
 func (f CsvFlag) GetValues(ctx *cli.Context) ([]string, error) {
 	val := ctx.String(f.Name)
 	selection := strings.Split(val, ",")

cmd/notifications/mark_as.go

@@ -35,7 +35,7 @@ var CmdNotificationsMarkRead = cli.Command{
 	},
 }
 
-// RunNotificationsMarkUnread will mark notifications as unread.
+// CmdNotificationsMarkUnread will mark notifications as unread.
 var CmdNotificationsMarkUnread = cli.Command{
 	Name:        "unread",
 	Aliases:     []string{"u"},
@@ -56,7 +56,7 @@ var CmdNotificationsMarkUnread = cli.Command{
 	},
 }
 
-// RunNotificationsMarkPinned will mark notifications as unread.
+// CmdNotificationsMarkPinned will mark notifications as unread.
 var CmdNotificationsMarkPinned = cli.Command{
 	Name:        "pin",
 	Aliases:     []string{"p"},
@@ -77,7 +77,7 @@ var CmdNotificationsMarkPinned = cli.Command{
 	},
 }
 
-// RunNotificationsUnpin will mark pinned notifications as unread.
+// CmdNotificationsUnpin will mark pinned notifications as unread.
 var CmdNotificationsUnpin = cli.Command{
 	Name:        "unpin",
 	Usage:       "Unpin all pinned or a specific notification",