issue-6: make "all" the default for modules"

michaeljon · May 17, 2022 · ac22a74 · ac22a74
1 parent e008268
commit ac22a74
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/Modules/ModuleFactory.cs b/Modules/ModuleFactory.cs
@@ -21,7 +21,7 @@ public static class ModuleFactory
 
         public static IEnumerable<IQcModule> Create(CliOptions settings)
         {
-            if (settings.ModuleNames.First() == "all")
+            if (settings.ModuleNames.Any() == false || settings.ModuleNames.First() == "all")
             {
                 settings.ModuleNames = moduleMap.Keys;
                 return moduleMap.Values;

diff --git a/Utils/CliOptions.cs b/Utils/CliOptions.cs
@@ -31,7 +31,7 @@ public class CliOptions
         [Option('z', "zipped", Required = false, HelpText = "Assume input file is gzipped.")]
         public bool Zipped { get; set; }
 
-        [Option('m', "modules", Required = true, Min = 1, HelpText = "Space-separated list of modules to run, or 'all'.")]
+        [Option('m', "modules", Required = false, HelpText = "Space-separated list of modules to run, or 'all'.")]
         public IEnumerable<string> ModuleNames { get; set; }
 
         public static CliOptions Settings