Address review comments

rr-wfm · Oct 1, 2024 · b692836 · b692836
1 parent 94d1144
commit b692836
Show file tree

Hide file tree

Showing 5 changed files with 5 additions and 5 deletions.
diff --git a/src/DacpacTool/BuildOptions.cs b/src/DacpacTool/BuildOptions.cs
@@ -22,7 +22,7 @@ public class BuildOptions : BaseOptions
 
         public bool RunCodeAnalysis { get; set; }
         public string CodeAnalysisRules { get; set; }
-        public FileInfo[] CodeAnalysisAssembly { get; set; }
+        public FileInfo[] CodeAnalysisAssemblies { get; set; }
         public bool WarnAsError { get; set; }
         public string SuppressWarnings { get; set; }
         public FileInfo SuppressWarningsListFile { get; set; }

diff --git a/src/DacpacTool/Program.cs b/src/DacpacTool/Program.cs
@@ -33,7 +33,7 @@ static async Task<int> Main(string[] args)
 
                 new Option<bool>(new string[] { "--runcodeanalysis", "-an" }, "Run static code analysis"),
                 new Option<string>(new string[] { "--codeanalysisrules", "-ar" }, "List of rules to suppress in format '-Microsoft.Rules.Data.SR0001;-Microsoft.Rules.Data.SR0008'"),
-                new Option<FileInfo[]>(new string[] { "--codeanalysisassembly", "-aa" }, "Custom code analysis rule assembly to use"),
+                new Option<FileInfo[]>(new string[] { "--codeanalysisassemblies", "-aa" }, "Custom code analysis rule assemblies to use"),
 
                 new Option<bool>(new string[] { "--warnaserror" }, "Treat T-SQL Warnings As Errors"),
                 new Option<bool>(new string[] { "--generatecreatescript", "-gcs" }, "Generate create script for package"),
@@ -207,7 +207,7 @@ private static int BuildDacpac(BuildOptions options)
             {
                 var analyzer = new PackageAnalyzer(new ActualConsole(), options.CodeAnalysisRules);
 
-                analyzer.Analyze(packageBuilder.Model, options.Output, options.CodeAnalysisAssembly ?? Array.Empty<FileInfo>());
+                analyzer.Analyze(packageBuilder.Model, options.Output, options.CodeAnalysisAssemblies ?? Array.Empty<FileInfo>());
             }
 
             return 0;

diff --git a/src/MSBuild.Sdk.SqlProj.Templates/templates/sqlproj/sqlproj.csproj b/src/MSBuild.Sdk.SqlProj.Templates/templates/sqlproj/sqlproj.csproj
@@ -8,6 +8,7 @@
 
     <ItemGroup>
         <!-- These packages adds additional code analysis rules -->
+        <!-- We recommend using these, but they can be removed if desired -->
         <PackageReference Include="ErikEJ.DacFX.SqlServer.Rules" Version="1.1.0" />
         <PackageReference Include="ErikEJ.DacFX.TSQLSmellSCA" Version="1.1.0" />
     </ItemGroup>

diff --git a/test/DacpacTool.Tests/PackageAnalyzerTests.cs b/test/DacpacTool.Tests/PackageAnalyzerTests.cs
@@ -160,7 +160,6 @@ private FileInfo[] CollectAssemblyPaths()
         {
             var result = new List<FileInfo>();
             var path = Path.GetDirectoryName(Path.Combine(System.Reflection.Assembly.GetAssembly(typeof(PackageAnalyzerTests)).Location));
-            result.Add(new FileInfo(Path.Combine(path, "SqlServer.Dac.dll")));
             result.Add(new FileInfo(Path.Combine(path, "SqlServer.Rules.dll")));
             result.Add(new FileInfo(Path.Combine(path, "TSQLSmellSCA.dll")));
 

diff --git a/test/TestProjectWithAnalyzers/TestProjectWithAnalyzers.csproj b/test/TestProjectWithAnalyzers/TestProjectWithAnalyzers.csproj
@@ -9,7 +9,7 @@
   </PropertyGroup>
 
   <ItemGroup>
-      <PackageReference Include="ErikEJ.DacFX.SqlServer.Rules" Version="1.0.1" />
+      <PackageReference Include="ErikEJ.DacFX.SqlServer.Rules" Version="1.1.0" />
   </ItemGroup>
 
   <Import Project="$(MSBuildThisFileDirectory)..\..\src\MSBuild.Sdk.SqlProj\Sdk\Sdk.targets" />