Merge pull request #238 from nellh/variadic-options-fix

fix: Use string[] options for datasetTypes and blacklistModalities

Author: effigies

changelog.d/20250826_135735_nell_variadic_options_fix.md

@@ -0,0 +1,47 @@
+<!--
+A new scriv changelog fragment.
+
+Uncomment the section that is right (remove the HTML comment wrapper).
+For top level release notes, leave all the headers commented out.
+-->
+
+<!--
+### Added
+
+- A bullet item for the Added category.
+
+-->
+### Changed
+
+- `--datasetTypes` and `--blacklistModalities` options now accept a string array (`--datasetTypes raw,derivative`) and can be combined.
+
+<!--
+### Fixed
+
+- A bullet item for the Fixed category.
+
+-->
+<!--
+### Deprecated
+
+- A bullet item for the Deprecated category.
+
+-->
+<!--
+### Removed
+
+- A bullet item for the Removed category.
+
+-->
+<!--
+### Security
+
+- A bullet item for the Security category.
+
+-->
+<!--
+### Infrastructure
+
+- A bullet item for the Infrastructure category.
+
+-->

src/setup/options.ts

@@ -76,13 +76,13 @@ export const validateCommand: Command<void, void, any, string[], void> = new Com
   )
   .type('datasetType', datasetType)
   .option(
-    '--datasetTypes <...datasetTypes:datasetType>',
+    '--datasetTypes <datasetTypes:datasetType[]>',
     'Permitted dataset types to validate against (default: all)',
     { default: [] as string[] },
   )
   .type('modality', modalityType)
   .option(
-    '--blacklistModalities <...modalities:modality>',
+    '--blacklistModalities <modalities:modality[]>',
     'Array of modalities to error on if detected.',
     { default: [] as string[] },
   )