diff options
| author | Taras Madan <tarasmadan@google.com> | 2024-09-16 14:29:26 +0200 |
|---|---|---|
| committer | Taras Madan <tarasmadan@google.com> | 2024-09-30 10:00:18 +0000 |
| commit | 179f40292f64debb6b3ea2fe5e950f62f5a05a18 (patch) | |
| tree | 664326084a8dee3bbe8847cf8196fd0b9417dee4 /pkg/covermerger/bq_csv_reader.go | |
| parent | ba29ff75d05fca62258d71ffc063e70f0d7b8b09 (diff) | |
pkg/validator: target value is not empty by default
The consistency make the code more predictable.
We want all the values to be non-empty.
If "empty" is an option - use validator.AnyOk(validator.EmptyStr(target), otherCheck(target)).
Diffstat (limited to 'pkg/covermerger/bq_csv_reader.go')
| -rw-r--r-- | pkg/covermerger/bq_csv_reader.go | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/pkg/covermerger/bq_csv_reader.go b/pkg/covermerger/bq_csv_reader.go index 60f6e5829..187da1812 100644 --- a/pkg/covermerger/bq_csv_reader.go +++ b/pkg/covermerger/bq_csv_reader.go @@ -34,7 +34,7 @@ func MakeBQCSVReader() *bqCSVReader { func (r *bqCSVReader) InitNsRecords(ctx context.Context, ns, filePath, commit string, from, to civil.Date) error { if err := validator.AnyError("input validation failed", validator.NamespaceName(ns), - validator.KernelFilePath(filePath), + validator.AnyOk(validator.EmptyStr(filePath), validator.KernelFilePath(filePath)), validator.AnyOk(validator.EmptyStr(commit), validator.CommitHash(commit)), ); err != nil { return err |