style: fix warnings based on new coding-rules

Author: Per Kops

src/Atc.CodingRules.AnalyzerProviders/Models/AnalyzerProviderBaseRuleData.cs

@@ -14,7 +14,7 @@ public AnalyzerProviderBaseRuleData(string name)
     public string Name { get; set; } = string.Empty;
 
     [SuppressMessage("Usage", "CA2227:Collection properties should be read only", Justification = "OK.")]
-    public ICollection<Rule> Rules { get; set; } = new List<Rule>();
+    public ICollection<Rule> Rules { get; set; } = [];
 
     public string? ExceptionMessage { get; set; }
 

src/Atc.CodingRules.Updater.CLI/Commands/DescriptionAttributes/SupportedProjectTargetTypeDescriptionAttribute.cs

@@ -8,7 +8,7 @@ public override string Description
         get
         {
             var defaultValue = new OptionsFile().ProjectTarget;
-            var values = Enum.GetNames(typeof(SupportedProjectTargetType))
+            var values = Enum.GetNames<SupportedProjectTargetType>()
                 .Select(enumValue => enumValue.Equals(defaultValue.ToString(), StringComparison.Ordinal)
                     ? $"{enumValue} (default)"
                     : enumValue)

src/Atc.CodingRules.Updater.CLI/Commands/RunCommand.cs

@@ -140,13 +140,10 @@ private static async Task<OptionsFile> GetOptionsFromFileAndUserArguments(
             buildFile = settings.BuildFile.Value;
         }
 
-        if (!string.IsNullOrEmpty(buildFile))
-        {
-            return buildFile.Contains(':', StringComparison.Ordinal)
+        return !string.IsNullOrEmpty(buildFile)
+            ? buildFile.Contains(':', StringComparison.Ordinal)
                 ? new FileInfo(buildFile)
-                : new FileInfo(Path.Combine(projectPath.FullName, buildFile));
-        }
-
-        return null;
+                : new FileInfo(Path.Combine(projectPath.FullName, buildFile))
+            : null;
     }
 }

src/Atc.CodingRules.Updater.CLI/Commands/Settings/ProjectBaseCommandSettings.cs

@@ -14,14 +14,11 @@ public class ProjectBaseCommandSettings : BaseCommandSettings
     public override ValidationResult Validate()
     {
         var validationResult = base.Validate();
-        if (!validationResult.Successful)
-        {
-            return validationResult;
-        }
-
-        return string.IsNullOrEmpty(ProjectPath)
-            ? ValidationResult.Error("ProjectPath is missing.")
-            : ValidationResult.Success();
+        return !validationResult.Successful
+            ? validationResult
+            : string.IsNullOrEmpty(ProjectPath)
+                ? ValidationResult.Error("ProjectPath is missing.")
+                : ValidationResult.Success();
     }
 
     internal string GetOptionsPath()

src/Atc.CodingRules.Updater.CLI/Models/Options/OptionsFolderMappings.cs

@@ -3,7 +3,7 @@ namespace Atc.CodingRules.Updater.CLI.Models.Options;
 public class OptionsFolderMappings
 {
     [SuppressMessage("Usage", "CA2227:Collection properties should be read only", Justification = "OK.")]
-    public IList<string> Paths { get; set; } = new List<string>();
+    public IList<string> Paths { get; set; } = [];
 
     public override string ToString()
         => $"{nameof(Paths)}.Count: {Paths.Count}";

src/Atc.CodingRules.Updater.CLI/ProjectHelper.cs

@@ -140,24 +140,17 @@ private static ProjectFrameworkType DetermineProjectFrameworkType(
                 Path.GetFileNameWithoutExtension(csProjFile.Name),
                 StringComparison.OrdinalIgnoreCase));
 
-        if (optionsProjectFrameworkMapping is not null)
+        projectFrameworkType = optionsProjectFrameworkMapping?.Type ?? projectType switch
         {
-            projectFrameworkType = optionsProjectFrameworkMapping.Type;
-        }
-        else
-        {
-            projectFrameworkType = projectType switch
-            {
-                DotnetProjectType.AzureFunctionApp => ProjectFrameworkType.AzureFunctions,
-                DotnetProjectType.BlazorServerApp or DotnetProjectType.BlazorWAsmApp => ProjectFrameworkType.Blazor,
-                DotnetProjectType.CliApp => ProjectFrameworkType.Cli,
-                DotnetProjectType.MauiApp => ProjectFrameworkType.Maui,
-                DotnetProjectType.WinFormApp => ProjectFrameworkType.WinForms,
-                DotnetProjectType.WpfApp or DotnetProjectType.WpfLibrary => ProjectFrameworkType.Wpf,
-                DotnetProjectType.WebApi => ProjectFrameworkType.WebApi,
-                _ => projectFrameworkType,
-            };
-        }
+            DotnetProjectType.AzureFunctionApp => ProjectFrameworkType.AzureFunctions,
+            DotnetProjectType.BlazorServerApp or DotnetProjectType.BlazorWAsmApp => ProjectFrameworkType.Blazor,
+            DotnetProjectType.CliApp => ProjectFrameworkType.Cli,
+            DotnetProjectType.MauiApp => ProjectFrameworkType.Maui,
+            DotnetProjectType.WinFormApp => ProjectFrameworkType.WinForms,
+            DotnetProjectType.WpfApp or DotnetProjectType.WpfLibrary => ProjectFrameworkType.Wpf,
+            DotnetProjectType.WebApi => ProjectFrameworkType.WebApi,
+            _ => projectFrameworkType,
+        };
 
         return projectFrameworkType;
     }

src/Atc.CodingRules.Updater/EditorConfigHelper.cs

@@ -333,7 +333,7 @@ private static List<Tuple<string, List<string>>> ExtractContentCustomParts(
                     }
 
                     workingOnCustomHeader = nextLine.Substring(CustomSectionHeaderPrefix.Length).Trim();
-                    workingOnCustomLines = new List<string>();
+                    workingOnCustomLines = [];
                 }
             }
 

src/Atc.CodingRules.Updater/Extensions/StringExtensions.cs

@@ -4,7 +4,7 @@ namespace System;
 
 public static class StringExtensions
 {
-    private static readonly string[] LineBreaks = { "\r\n", "\r", "\n" };
+    private static readonly string[] LineBreaks = ["\r\n", "\r", "\n"];
 
     public static string TrimEndForEmptyLines(
         this string value)
@@ -25,7 +25,7 @@ public static string TrimEndForEmptyLines(
     public static Collection<KeyValueItem> GetKeyValues(
         this string value)
         => string.IsNullOrEmpty(value)
-            ? new Collection<KeyValueItem>()
+            ? []
             : value
                 .Split(LineBreaks, StringSplitOptions.RemoveEmptyEntries)
                 .GetKeyValues();

src/Atc.CodingRules.Updater/FileHelper.cs

@@ -1,4 +1,3 @@
-// ReSharper disable ConvertIfStatementToReturnStatement
 namespace Atc.CodingRules.Updater;
 
 public static class FileHelper
@@ -83,13 +82,8 @@ public static bool AreFilesEqual(
             return false;
         }
 
-        if (headerLinesA.Find(x => x.StartsWith("# Distribution", StringComparison.CurrentCultureIgnoreCase)) !=
-            headerLinesB.Find(x => x.StartsWith("# Distribution", StringComparison.CurrentCultureIgnoreCase)))
-        {
-            return false;
-        }
-
-        return true;
+        return headerLinesA.Find(x => x.StartsWith("# Distribution", StringComparison.CurrentCultureIgnoreCase)) ==
+               headerLinesB.Find(x => x.StartsWith("# Distribution", StringComparison.CurrentCultureIgnoreCase));
     }
 
     public static bool ContainsEditorConfigFile(