Fix Meziantou.Analyzer dependency (#634)

Author: sebastienros

src/Directory.Packages.props

@@ -9,9 +9,12 @@
     <PackageVersion Include="Npgsql" Version="9.0.3" />
     <PackageVersion Include="MySqlConnector" Version="2.4.0" />
     <PackageVersion Include="Dapper.StrongName" Version="$(DapperStrongNamePackageVersion)" />
-    <PackageVersion Include="Meziantou.Analyzer" Version="2.0.162" />    
     <PackageVersion Include="Microsoft.Extensions.DependencyInjection.Abstractions" Version="8.0.0" />
     <PackageVersion Include="Microsoft.Extensions.Logging.Abstractions" Version="8.0.0" />
     <PackageVersion Include="Microsoft.SourceLink.GitHub" Version="8.0.0" />
   </ItemGroup>
+
+  <ItemGroup>
+    <GlobalPackageReference Include="Meziantou.Analyzer" Version="2.0.177" />
+  </ItemGroup>
 </Project>

src/YesSql.Abstractions/YesSql.Abstractions.csproj

@@ -1,6 +1,5 @@
 <Project Sdk="Microsoft.NET.Sdk">
     <ItemGroup>
-        <PackageReference Include="Meziantou.Analyzer" />
         <!-- Latest minimum LTS version. We don't want to force a higher version to applications -->
         <PackageReference Include="Microsoft.Extensions.Logging.Abstractions" />
     </ItemGroup>

src/YesSql.Core/YesSql.Core.csproj

@@ -5,7 +5,6 @@
 
     <ItemGroup>
         <PackageReference Include="Dapper.StrongName" />
-        <PackageReference Include="Meziantou.Analyzer" />
         <PackageReference Include="Microsoft.Extensions.DependencyInjection.Abstractions" />
     </ItemGroup>
 

src/YesSql.Filters.Abstractions/Builders/TermEngineBuilder.cs

@@ -6,7 +6,7 @@ namespace YesSql.Filters.Abstractions.Builders
 {
     public abstract class TermEngineBuilder<T, TTermOption>  where TTermOption : TermOption
     {
-        public TermEngineBuilder(string name)
+        protected TermEngineBuilder(string name)
         {
             Name = name;
         }

src/YesSql.Filters.Abstractions/Builders/UnaryEngineBuilder.cs

@@ -15,7 +15,7 @@ public abstract class UnaryEngineBuilder<T, TTermOption> : OperatorEngineBuilder
 
         protected TTermOption _termOption;
 
-        public UnaryEngineBuilder(TTermOption termOption)
+        protected UnaryEngineBuilder(TTermOption termOption)
         {
             _termOption = termOption;
         }

src/YesSql.Filters.Abstractions/Nodes/OperatorNodes.cs

@@ -41,7 +41,7 @@ public override string ToString()
                     OperateNodeQuotes.None => Value,
                     OperateNodeQuotes.Double => $"\"{Value}\"",
                     OperateNodeQuotes.Single => $"\'{Value}\'",
-                    _ => throw new NotImplementedException()
+                    _ => throw new NotSupportedException()
                 };
             }
             else

src/YesSql.Filters.Abstractions/Nodes/TermNodes.cs

@@ -6,7 +6,7 @@ namespace YesSql.Filters.Abstractions.Nodes
 {
     public abstract class TermNode : FilterNode
     {
-        public TermNode(string termName)
+        protected TermNode(string termName)
         {
             TermName = termName;
         }
@@ -18,7 +18,7 @@ public override TResult Accept<TArgument, TResult>(IFilterVisitor<TArgument, TRe
 
     public abstract class TermOperationNode : TermNode
     {
-        public TermOperationNode(string termName, OperatorNode operation) : base(termName)
+        protected TermOperationNode(string termName, OperatorNode operation) : base(termName)
         {
             Operation = operation;
         }
@@ -58,7 +58,7 @@ public override string ToString()
 
     public abstract class CompoundTermNode : TermNode
     {
-        public CompoundTermNode(string termName) : base(termName)
+        protected CompoundTermNode(string termName) : base(termName)
         {
         }
 

src/YesSql.Filters.Abstractions/Services/FilterExecutionContext.cs

@@ -2,7 +2,7 @@ namespace YesSql.Filters.Abstractions.Services
 {
     public abstract class FilterExecutionContext<T>
     {
-        public FilterExecutionContext(T item)
+        protected FilterExecutionContext(T item)
         {
             Item = item;
         }

src/YesSql.Filters.Abstractions/Services/FilterResult.cs

@@ -11,12 +11,12 @@ public abstract class FilterResult<T, TTermOption> : IEnumerable<TermNode> where
 
         protected Dictionary<string, TermNode> _terms = new Dictionary<string, TermNode>(StringComparer.OrdinalIgnoreCase);
 
-        public FilterResult(IReadOnlyDictionary<string, TTermOption> termOptions)
+        protected FilterResult(IReadOnlyDictionary<string, TTermOption> termOptions)
         {
             TermOptions = termOptions;
         }
 
-        public FilterResult(IReadOnlyList<TermNode> terms, IReadOnlyDictionary<string, TTermOption> termOptions)
+        protected FilterResult(IReadOnlyList<TermNode> terms, IReadOnlyDictionary<string, TTermOption> termOptions)
         {
             TermOptions = termOptions;
 

src/YesSql.Filters.Abstractions/Services/TermOption.cs

@@ -5,7 +5,7 @@ namespace YesSql.Filters.Abstractions.Services
 {
     public abstract class TermOption
     {
-        public TermOption(string name)
+        protected TermOption(string name)
         {
             Name = name;
         }