Run nilaway again (#107)

* Run nilaway again

* Remove outdated ref to pflag

Taskfile.yml

@@ -4,7 +4,6 @@ version: "3"
 
 vars:
   COV_DATA: coverage.out
-  PACKAGE: github.com/FollowTheProcess/cli
 
 tasks:
   default:
@@ -63,7 +62,7 @@ tasks:
         msg: nilaway not installed, see https://github.com/uber-go/nilaway
     cmds:
       - golangci-lint run --fix
-      # - nilaway -include-pkgs="{{.PACKAGE}}" ./... # nilaway doesn't work with go 1.23 yet
+      - nilaway ./...
 
   docs:
     desc: Render the pkg docs locally

command_test.go

@@ -810,7 +810,9 @@ func TestDuplicateSubCommands(t *testing.T) {
 	)
 
 	test.Err(t, err)
-	test.Equal(t, err.Error(), `subcommand "sub1" already defined`)
+	if err != nil {
+		test.Equal(t, err.Error(), `subcommand "sub1" already defined`)
+	}
 }
 
 func TestCommandNoRunNoSub(t *testing.T) {

internal/flag/flag.go

@@ -119,7 +119,7 @@ func (f Flag[T]) Get() T {
 }
 
 // String implements [fmt.Stringer] for a [Flag], and also implements the String
-// part of [pflag.Value], allowing a flag to print itself.
+// part of [Value], allowing a flag to print itself.
 func (f Flag[T]) String() string { //nolint:gocyclo // No other way of doing this realistically
 	if f.value == nil {
 		return ""