Ensure parser configs are json-able

urfave · Jul 2, 2023 · da1ca8b · da1ca8b
1 parent e20e7ba
commit da1ca8b
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/example_test.go b/example_test.go
@@ -62,6 +62,10 @@ func ExampleParserConfig_simple() {
 		"hello", "-a=from", "the", "ether", "sub", "marge", "patty", "selma", "-b",
 	}
 
+	if _, err := json.Marshal(pCfg.Prog); err != nil {
+		log.Fatal(err)
+	}
+
 	pt, err := argh.ParseArgs(os.Args, pCfg)
 	if err != nil {
 		argh.PrintParserError(os.Stderr, err)

diff --git a/parser_config.go b/parser_config.go
@@ -72,7 +72,7 @@ type CommandConfig struct {
 	Flags      *Flags
 	Commands   *Commands
 
-	On func(CommandFlag)
+	On func(CommandFlag) `json:"-"`
 }
 
 func (cCfg *CommandConfig) init() {
@@ -137,7 +137,7 @@ type FlagConfig struct {
 	Persist    bool
 	ValueNames []string
 
-	On func(CommandFlag)
+	On func(CommandFlag) `json:"-"`
 }
 
 type Flags struct {