Minor fixes of dotnet code analysis errors

Author: ckadluba

sample/CombinedConfigDemo/Program.cs

@@ -49,7 +49,7 @@ public static void Main()
                     columnOptionsSection: columnOptionsSection)
                 .CreateLogger();
 
-            Log.Information("Hello {Name} from thread {ThreadId}", Environment.GetEnvironmentVariable("USERNAME"), Thread.CurrentThread.ManagedThreadId);
+            Log.Information("Hello {Name} from thread {ThreadId}", Environment.GetEnvironmentVariable("USERNAME"), Environment.CurrentManagedThreadId);
 
             Log.Warning("No coins remain at position {@Position}", new { Lat = 25, Long = 134 });
 

sample/CustomLogEventFormatterDemo/Program.cs

@@ -57,7 +57,7 @@ public static void Main()
             {
                 Log.Debug("Getting started");
 
-                Log.Information("Hello {Name} from thread {ThreadId}", Environment.GetEnvironmentVariable("USERNAME"), Thread.CurrentThread.ManagedThreadId);
+                Log.Information("Hello {Name} from thread {ThreadId}", Environment.GetEnvironmentVariable("USERNAME"), Environment.CurrentManagedThreadId);
 
                 Log.Warning("No coins remain at position {@Position}", new { Lat = 25, Long = 134 });
 

test/Serilog.Sinks.MSSqlServer.Tests/GlobalSuppressions.cs

@@ -8,4 +8,3 @@
 [assembly: SuppressMessage("Globalization", "CA1303:Do not pass literals as localized parameters", Justification = "Supplying string literals and not using resources is accepted within this project.", Scope = "namespaceanddescendants", Target = "Serilog.Sinks.MSSqlServer.Tests")]
 [assembly: SuppressMessage("Naming", "CA1707:Identifiers should not contain underscores", Justification = "Member names must match SQL server master DB objects.", Scope = "type", Target = "~T:Serilog.Sinks.MSSqlServer.Tests.TestUtils.sp_pkey")]
 [assembly: SuppressMessage("Naming", "CA1707:Identifiers should not contain underscores", Justification = "Member names must match SQL server master DB objects.", Scope = "type", Target = "~T:Serilog.Sinks.MSSqlServer.Tests.TestUtils.SysIndex_CCI")]
-[assembly: SuppressMessage("Globalization", "CA1307:Specify StringComparison", Justification = "In net452 there is no Contains() overload that accepts a StringComparison specifier", Scope = "namespaceanddescendants", Target = "Serilog.Sinks.MSSqlServer.Tests")]

test/Serilog.Sinks.MSSqlServer.Tests/Sinks/MSSqlServer/Output/AdditionalColumnDataGeneratorTests.cs

@@ -13,14 +13,12 @@ namespace Serilog.Sinks.MSSqlServer.Tests.Output
     [Trait(TestCategory.TraitName, TestCategory.Unit)]
     public class AdditionalColumnDataGeneratorTests
     {
-        private readonly Dictionary<string, LogEventPropertyValue> _properties;
-        private Mock<IColumnSimplePropertyValueResolver> _columnSimplePropertyValueResolver;
-        private Mock<IColumnHierarchicalPropertyValueResolver> _columnHierarchicalPropertyValueResolver;
-        private AdditionalColumnDataGenerator _sut;
+        private readonly Mock<IColumnSimplePropertyValueResolver> _columnSimplePropertyValueResolver;
+        private readonly Mock<IColumnHierarchicalPropertyValueResolver> _columnHierarchicalPropertyValueResolver;
+        private readonly AdditionalColumnDataGenerator _sut;
 
         public AdditionalColumnDataGeneratorTests()
         {
-            _properties = new Dictionary<string, LogEventPropertyValue>();
             _columnSimplePropertyValueResolver = new Mock<IColumnSimplePropertyValueResolver>();
             _columnHierarchicalPropertyValueResolver = new Mock<IColumnHierarchicalPropertyValueResolver>();
             _sut = new AdditionalColumnDataGenerator(_columnSimplePropertyValueResolver.Object,