fix: make -skip-validation flag not depend on -validate flag being pr…

…esent

right now `-skip-validation` is only considered if the -validate flag is
present; the name of the flag implies it being a boolean flag and it is
unintuitive that it needs another flag to be present referencing a
config file that will be ignored

cmd/achcli/main.go

@@ -93,6 +93,13 @@ func main() {
 }
 
 func readValidationOpts(path string) *ach.ValidateOpts {
+	var opts ach.ValidateOpts
+
+	if *flagSkipValidation {
+		opts.SkipAll = true
+		return &opts
+	}
+
 	if path != "" {
 		// read config file
 		bs, readErr := os.ReadFile(path)
@@ -101,14 +108,10 @@ func readValidationOpts(path string) *ach.ValidateOpts {
 			os.Exit(1)
 		}
 
-		var opts ach.ValidateOpts
 		if err := json.Unmarshal(bs, &opts); err != nil {
 			fmt.Printf("ERROR: unmarshal of validate opts failed: %v\n", err)
 			os.Exit(1)
 		}
-		if *flagSkipValidation {
-			opts.SkipAll = true
-		}
 		return &opts
 	}
 	return nil