Merge pull request #310 from atc-net/feature/spectre-console

spectre console version argument fix
atc-net · Apr 2, 2024 · 416612c · 416612c
2 parents fb401ce + 2cb5cca
commit 416612c
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 5 deletions.
diff --git a/atc-coding-rules-updater.ps1 b/atc-coding-rules-updater.ps1
@@ -9,4 +9,4 @@ atc-coding-rules-updater `
     run `
     -p $currentPath `
     --optionsPath $currentPath'\atc-coding-rules-updater.json' `
-    -v
+    --verbose
diff --git a/src/Atc.Console.Spectre/CommandConstants.cs b/src/Atc.Console.Spectre/CommandConstants.cs
@@ -4,8 +4,8 @@ public static class CommandConstants
 {
     public const string ArgumentShortHelp = "-h";
     public const string ArgumentLongHelp = "--help";
+    public const string ArgumentShortVersion = "-v";
     public const string ArgumentLongVersion = "--version";
-    public const string ArgumentShortVerbose = "-v";
     public const string ArgumentLongVerbose = "--verbose";
 
     public const string NameOptionsFile = "options-file";

diff --git a/src/Atc.Console.Spectre/CommandSettings/BaseCommandSettings.cs b/src/Atc.Console.Spectre/CommandSettings/BaseCommandSettings.cs
@@ -2,7 +2,7 @@ namespace Atc.Console.Spectre.CommandSettings;
 
 public class BaseCommandSettings : global::Spectre.Console.Cli.CommandSettings
 {
-    [CommandOption($"{CommandConstants.ArgumentShortVerbose}|{CommandConstants.ArgumentLongVerbose}")]
+    [CommandOption($"{CommandConstants.ArgumentLongVerbose}")]
     [Description("Use verbose for more debug/trace information")]
     public bool Verbose { get; set; }
 

diff --git a/src/Atc.Console.Spectre/Helpers/ProgramCsHelper.cs b/src/Atc.Console.Spectre/Helpers/ProgramCsHelper.cs
@@ -17,8 +17,7 @@ public static void SetMinimumLogLevelIfNeeded(
             throw new ArgumentNullException(nameof(consoleLoggerConfiguration));
         }
 
-        if (args.Any(x => x.Equals(CommandConstants.ArgumentShortVerbose, StringComparison.OrdinalIgnoreCase)) ||
-            args.Any(x => x.Equals(CommandConstants.ArgumentLongVerbose, StringComparison.OrdinalIgnoreCase)))
+        if (args.Any(x => x.Equals(CommandConstants.ArgumentLongVerbose, StringComparison.OrdinalIgnoreCase)))
         {
             consoleLoggerConfiguration.MinimumLogLevel = LogLevel.Trace;
         }
-Original file line number
+Diff line change
@@ Expand Up / @@ -9,4 +9,4 @@ atc-coding-rules-updater ` @@
         run `
         -p $currentPath `
         --optionsPath $currentPath'\atc-coding-rules-updater.json' `
-        -v
+        --verbose