Deprecate options (#299)

raviqqe · Apr 19, 2023 · 9cd738f · 9cd738f
1 parent e385569
commit 9cd738f
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 6 deletions.
diff --git a/.snapshots/TestHelp b/.snapshots/TestHelp
@@ -21,9 +21,11 @@ Application Options:
       --header=<header>...                  Custom headers
   -f, --ignore-fragments                    Ignore URL fragments
       --format=[text|json|junit]            Output format (default: text)
-      --json                                Output results in JSON
+      --json                                Output results in JSON (deprecated)
       --experimental-verbose-json           Include successful results in JSON
+                                            (deprecated)
       --junit                               Output results as JUnit XML file
+                                            (deprecated)
   -r, --max-redirections=<count>            Maximum number of redirections
                                             (default: 64)
       --rate-limit=<rate>                   Max requests per second

diff --git a/arguments.go b/arguments.go
@@ -22,11 +22,12 @@ type arguments struct {
 	// TODO Remove a short option.
 	IgnoreFragments bool   `short:"f" long:"ignore-fragments" description:"Ignore URL fragments"`
 	Format          string `long:"format" description:"Output format" default:"text" choice:"text" choice:"json" choice:"junit"`
-	// TODO Merge text, JSON, and JUnit XML format options into --format.
-	JSONOutput bool `long:"json" description:"Output results in JSON"`
-	// TODO Integrate this option into --verbose.
-	VerboseJSON         bool   `long:"experimental-verbose-json" description:"Include successful results in JSON"`
-	JUnitOutput         bool   `long:"junit" description:"Output results as JUnit XML file"`
+	// TODO Remove this option.
+	JSONOutput bool `long:"json" description:"Output results in JSON (deprecated)"`
+	// TODO Remove this option.
+	VerboseJSON bool `long:"experimental-verbose-json" description:"Include successful results in JSON (deprecated)"`
+	// TODO Remove this option.
+	JUnitOutput         bool   `long:"junit" description:"Output results as JUnit XML file (deprecated)"`
 	MaxRedirections     int    `short:"r" long:"max-redirections" value-name:"<count>" default:"64" description:"Maximum number of redirections"`
 	RateLimit           int    `long:"rate-limit" value-name:"<rate>" description:"Max requests per second"`
 	Timeout             int    `short:"t" long:"timeout" value-name:"<seconds>" default:"10" description:"Timeout for HTTP requests in seconds"`