Merge pull request #38 from launchrctl/76-avoid_recursive_discovery

davidferlay · web-flow · commit d63e8ed11840 · 2024-03-28T12:58:14.000+01:00
76: skip discover actions for version check
diff --git a/app.go b/app.go
@@ -139,6 +139,14 @@ func (app *appImpl) init() error {
 		}
 	}
 
+	// Skip discover actions if we check version.
+	args := os.Args[1:]
+	for i := 0; i < len(args); i++ {
+		if args[i] == "--version" {
+			return nil
+		}
+	}
+
 	// Discover actions.
 	for _, p := range getPluginByType[ActionDiscoveryPlugin](app) {
 		for _, fs := range app.GetRegisteredFS() {

Original file line number	Diff line number	Diff line change
`@@ -139,6 +139,14 @@ func (app *appImpl) init() error {`
`139`	`139`	`}`
`140`	`140`	`}`
`141`	`141`
	`142`	`+ // Skip discover actions if we check version.`
	`143`	`+ args := os.Args[1:]`
	`144`	`+ for i := 0; i < len(args); i++ {`
	`145`	`+ if args[i] == "--version" {`
	`146`	`+ return nil`
	`147`	`+ }`
	`148`	`+ }`
	`149`	`+`
`142`	`150`	`// Discover actions.`
`143`	`151`	`for _, p := range getPluginByType[ActionDiscoveryPlugin](app) {`
`144`	`152`	`for _, fs := range app.GetRegisteredFS() {`