Moved the logging options to the json file, meaning the cli is back to only having a single option to run. Updated readme

Author: ProgrammerAL

README.md

@@ -32,34 +32,43 @@ There are 2 ways to run this. As a .NET Tool installed on your machine, or downl
 - --config-file, -d
 	- Required
 	- Path to the file to use for config values when updating code
-- --output-file, -o
-  - Optional
-  - If this is set, it will be the file to write logs to, in addition to the console
-- --log-level, -l
-  - Optional
-  - Verbosity level to log. Valid values are: Verbose, Info, Warn, Error. Default value: verbose.
+- --help, -h
+  - Outputs CLI help
 
 ## Config File
 
 The config file holds all values to determine what changes to make. The reason this is separate from CLI input arguments is to let a developer store this config in different repos but have this .NET Tool installed globally on their machine. That makes it easy to let other developers run this tool with specific settings for each repository, while only needing to provide a single CLI input argument.
 
-Below are the list of properties in the config file. All fields are required.
+Below are the list of properties in the config file.
 
 - RootDirectory
+  - Required
 	- Root directory to run from. Code Updater will search all child directories within this for projects to update.
 - IgnorePatterns
+  - Required
 	- String to ignore within file paths when looking for projects to update. This is OS sensitive, so use \ as the path separator for Windows, and / as the path separator everywhere else. Eg: `\my-skip-path\` will ignore all projects that have the text `\my-skip-path\` within the full path. Note this example will only happen on Windows because that uses backslashes for path separators.- NpmBuildCommand
 - NpmBuildCommand
+  - Required
 	- After upgrading all of the code, this application will attempt to build all applications it updated. This option sets the npm command to run to do the build.
 	- Npm command to run to "compile" the npm directory. Default value is `publish`. Format run is: npm run <npmBuildCommand>.
 - DotNetTargetFramework
+  - Required
 	- Target Framework to set in all *.CsProj files
 - DotNetLangVersion
+  - Required
 	- C# language version to set in all *.CsProj files
 - EnableNetAnalyzers
+  - Required
 	- Boolean value to set the `EnableNetAnalyzers` csproj element to. If the `EnableNetAnalyzers` element does not exist in the project file, it will be added.
 - EnforceCodeStyleInBuild
+  - Required
 	- Boolean value to set the `EnforceCodeStyleInBuild` csproj element to. If the `EnableNetAnalyzers` element does not exist in the project file, it will be added.
+- OutputFile
+  - Optional
+  - If this is set, it will be the file to write logs to, in addition to the console
+- LogLevel
+  - Optional
+  - Verbosity level to log. Valid values are: Verbose, Info, Warn, Error. Default value: verbose.
 
 ### Example Config File
 

src/CodeUpdater/CodeUpdater/Options/CommandOptions.cs

@@ -13,10 +13,4 @@ public class CommandOptions
 {
     [Option(shortName: 'c', longName: "config-file", Required = true, HelpText = "Path to the file to use for config values when updating code")]
     public required string ConfigFile { get; set; }
-
-    [Option(shortName: 'o', longName: "output-file", Required = false, HelpText = "If this is set, it will be the file to write logs to, in addition to the console")]
-    public string? OutputFile { get; set; }
-
-    [Option(shortName: 'l', longName: "log-level", Required = false, HelpText = "Verbosity level to log. Valid values are: Verbose, Info, Warn, Error. Default value: verbose.")]
-    public string LogLevel { get; set; } = "verbose";
 }

src/CodeUpdater/CodeUpdater/Options/UpdateOptions.cs

@@ -55,4 +55,14 @@ public class UpdateOptions
     /// </summary>
     [Required]
     public required bool EnforceCodeStyleInBuild { get; set; }
+
+    /// <summary>
+    /// If this is set, it will be the file to write logs to, in addition to the console
+    /// </summary>
+    public string? OutputFile { get; set; }
+
+    /// <summary>
+    /// Verbosity level to log. Valid values are: Verbose, Info, Warn, Error. Default value: verbose.
+    /// </summary>
+    public string? LogLevel { get; set; } = "verbose";
 }

src/CodeUpdater/CodeUpdater/Program.cs

@@ -20,11 +20,10 @@ await Parser.Default.ParseArguments<CommandOptions>(args)
              await RunAsync(options);
          });
 
-static async ValueTask RunAsync(CommandOptions options)
+static async ValueTask RunAsync(CommandOptions commandOptions)
 {
-    var logger = SetupLogger(options);
-
-    var updateOptions = await LoadUpdateOptionsAsync(options.ConfigFile);
+    var updateOptions = await LoadUpdateOptionsAsync(commandOptions.ConfigFile);
+    var logger = SetupLogger(updateOptions);
 
     var runProcessHelper = new RunProcessHelper(logger);
     var workLocator = new WorkLocator(logger);
@@ -56,23 +55,23 @@ static async ValueTask RunAsync(CommandOptions options)
     OutputSummary(updateWork, csUpdates, npmUpdates, compileResults, logger);
 }
 
-static ILogger SetupLogger(CommandOptions options)
+static ILogger SetupLogger(UpdateOptions updateOptions)
 {
     var loggerConfig = new LoggerConfiguration()
         .WriteTo.Console();
 
-    if (!string.IsNullOrWhiteSpace(options.OutputFile))
+    if (!string.IsNullOrWhiteSpace(updateOptions.OutputFile))
     {
-        loggerConfig = loggerConfig.WriteTo.File(options.OutputFile);
+        loggerConfig = loggerConfig.WriteTo.File(updateOptions.OutputFile);
     }
 
-    if (string.IsNullOrWhiteSpace(options.LogLevel))
+    if (string.IsNullOrWhiteSpace(updateOptions.LogLevel))
     {
         loggerConfig = loggerConfig.MinimumLevel.Verbose();
     }
     else
     {
-        loggerConfig = options.LogLevel.ToLower() switch
+        loggerConfig = updateOptions.LogLevel.ToLower() switch
         {
             "verbose" => loggerConfig.MinimumLevel.Verbose(),
             "info" => loggerConfig.MinimumLevel.Information(),