More staticcheck fixes (#2007)

* more staticcheck

* more staticcheck

pkg/transformers/altkv.go

@@ -56,6 +56,7 @@ func transformerAltkvParseCLI(
 			return nil, cli.ErrHelpRequested
 		}
 		transformerAltkvUsage(os.Stderr)
+		//lint:ignore SA4004 loop is intentionally structured this way
 		return nil, fmt.Errorf("%s %s: option \"%s\" not recognized", "mlr", verbNameAltkv, opt)
 	}
 

pkg/transformers/check.go

@@ -59,6 +59,7 @@ func transformerCheckParseCLI(
 			return nil, cli.ErrHelpRequested
 
 		} else {
+			//lint:ignore SA4004 loop is intentionally structured this way
 			return nil, cli.VerbErrorf(verbNameCheck, "option \"%s\" not recognized", opt)
 		}
 	}

pkg/transformers/group_by.go

@@ -55,6 +55,7 @@ func transformerGroupByParseCLI(
 			return nil, cli.ErrHelpRequested
 
 		} else {
+			//lint:ignore SA4004 loop is intentionally structured this way
 			return nil, cli.VerbErrorf(verbNameGroupBy, "option \"%s\" not recognized", opt)
 		}
 	}

pkg/transformers/group_like.go

@@ -55,6 +55,7 @@ func transformerGroupLikeParseCLI(
 			return nil, cli.ErrHelpRequested
 
 		} else {
+			//lint:ignore SA4004 loop is intentionally structured this way
 			return nil, cli.VerbErrorf(verbNameGroupLike, "option \"%s\" not recognized", opt)
 		}
 	}

pkg/transformers/label.go

@@ -59,6 +59,7 @@ func transformerLabelParseCLI(
 			return nil, cli.ErrHelpRequested
 
 		} else {
+			//lint:ignore SA4004 loop is intentionally structured this way
 			return nil, cli.VerbErrorf(verbNameLabel, "option \"%s\" not recognized", opt)
 		}
 	}

pkg/transformers/latin1_to_utf8.go

@@ -57,6 +57,7 @@ func transformerLatin1ToUTF8ParseCLI(
 			return nil, cli.ErrHelpRequested
 
 		} else {
+			//lint:ignore SA4004 loop is intentionally structured this way
 			return nil, cli.VerbErrorf(verbNameLatin1ToUTF8, "option \"%s\" not recognized", opt)
 		}
 	}

pkg/transformers/nothing.go

@@ -55,6 +55,7 @@ func transformerNothingParseCLI(
 			return nil, cli.ErrHelpRequested
 		} else {
 			transformerNothingUsage(os.Stderr)
+			//lint:ignore SA4004 loop is intentionally structured this way
 			return nil, fmt.Errorf("%s %s: option \"%s\" not recognized", "mlr", verbNameNothing, opt)
 		}
 	}

pkg/transformers/regularize.go

@@ -56,6 +56,7 @@ func transformerRegularizeParseCLI(
 			return nil, cli.ErrHelpRequested
 
 		} else {
+			//lint:ignore SA4004 loop is intentionally structured this way
 			return nil, cli.VerbErrorf(verbNameRegularize, "option \"%s\" not recognized", opt)
 		}
 	}

pkg/transformers/remove_empty_columns.go

@@ -55,6 +55,7 @@ func transformerRemoveEmptyColumnsParseCLI(
 			return nil, cli.ErrHelpRequested
 
 		} else {
+			//lint:ignore SA4004 loop is intentionally structured this way
 			return nil, cli.VerbErrorf(verbNameRemoveEmptyColumns, "option \"%s\" not recognized", opt)
 		}
 	}

pkg/transformers/sec2gmtdate.go

@@ -58,6 +58,7 @@ func transformerSec2GMTDateParseCLI(
 			return nil, cli.ErrHelpRequested
 
 		} else {
+			//lint:ignore SA4004 loop is intentionally structured this way
 			return nil, cli.VerbErrorf(verbNameSec2GMTDate, "option \"%s\" not recognized", opt)
 		}
 	}

pkg/transformers/shuffle.go

@@ -58,6 +58,7 @@ func transformerShuffleParseCLI(
 			return nil, cli.ErrHelpRequested
 
 		} else {
+			//lint:ignore SA4004 loop is intentionally structured this way
 			return nil, cli.VerbErrorf(verbNameShuffle, "option \"%s\" not recognized", opt)
 		}
 	}

pkg/transformers/skip_trivial_records.go

@@ -55,6 +55,7 @@ func transformerSkipTrivialRecordsParseCLI(
 			return nil, cli.ErrHelpRequested
 
 		} else {
+			//lint:ignore SA4004 loop is intentionally structured this way
 			return nil, cli.VerbErrorf(verbNameSkipTrivialRecords, "option \"%s\" not recognized", opt)
 		}
 	}

pkg/transformers/tac.go

@@ -54,6 +54,7 @@ func transformerTacParseCLI(
 			return nil, cli.ErrHelpRequested
 
 		} else {
+			//lint:ignore SA4004 loop is intentionally structured this way
 			return nil, cli.VerbErrorf(verbNameTac, "option \"%s\" not recognized", opt)
 		}
 	}

pkg/transformers/utf8_to_latin1.go

@@ -57,6 +57,7 @@ func transformerUTF8ToLatin1ParseCLI(
 			return nil, cli.ErrHelpRequested
 
 		} else {
+			//lint:ignore SA4004 loop is intentionally structured this way
 			return nil, cli.VerbErrorf(verbNameUTF8ToLatin1, "option \"%s\" not recognized", opt)
 		}
 	}