Serilog 3 (#35)

Author: sungam3r

src/Destructurama.ByIgnoring.Tests/Support/DelegatingSink.cs

@@ -26,8 +26,5 @@ public DelegatingSink(Action<LogEvent> write)
         _write = write ?? throw new ArgumentNullException(nameof(write));
     }
 
-    public void Emit(LogEvent logEvent)
-    {
-        _write(logEvent);
-    }
+    public void Emit(LogEvent logEvent) => _write(logEvent);
 }

src/Destructurama.ByIgnoring/ByIgnoring/DestructureByIgnoringPolicy.cs

@@ -13,6 +13,7 @@
 // limitations under the License.
 
 using System.Collections.Concurrent;
+using System.Diagnostics.CodeAnalysis;
 using System.Linq.Expressions;
 using System.Reflection;
 using Serilog.Core;
@@ -37,7 +38,7 @@ public DestructureByIgnoringPolicy(Func<object, bool> handleDestructuringPredica
             throw new ArgumentOutOfRangeException(nameof(ignoredPropertyPredicates), "At least one ignore rule must be supplied");
     }
 
-    public bool TryDestructure(object value, ILogEventPropertyValueFactory propertyValueFactory, out LogEventPropertyValue? result)
+    public bool TryDestructure(object value, ILogEventPropertyValueFactory propertyValueFactory, [NotNullWhen(true)] out LogEventPropertyValue? result)
     {
         if (value == null || !_handleDestructuringPredicate(value))
         {
@@ -100,7 +101,7 @@ private static StructureValue BuildStructure(object value, ILogEventPropertyValu
     }
 
     private static LogEventPropertyValue BuildLogEventProperty(object propertyValue, ILogEventPropertyValueFactory propertyValueFactory)
-    {
-        return propertyValue == null ? new ScalarValue(null) : propertyValueFactory.CreatePropertyValue(propertyValue, destructureObjects: true);
-    }
+        => propertyValue == null
+            ? ScalarValue.Null
+            : propertyValueFactory.CreatePropertyValue(propertyValue, destructureObjects: true);
 }

src/Destructurama.ByIgnoring/Destructurama.ByIgnoring.csproj

@@ -6,10 +6,12 @@
     <Product>Serilog Lambda-Driven Destructuring Support</Product>
     <Description>Specify how complex types are logged to Serilog by excluding individual properties.</Description>
     <RootNamespace>Destructurama</RootNamespace>
+    <PolySharpIncludeRuntimeSupportedAttributes>true</PolySharpIncludeRuntimeSupportedAttributes>
   </PropertyGroup>
 
   <ItemGroup>
-    <PackageReference Include="Serilog" Version="2.8.0" />
+    <PackageReference Include="Serilog" Version="3.1.1" />
+    <PackageReference Include="PolySharp" Version="1.12.1" PrivateAssets="All" />
     <Using Remove="System.Net.Http"/>
     <InternalsVisibleTo Include="Destructurama.ByIgnoring.Tests, PublicKey=00240000048000009400000006020000002400005253413100040000010001000fe215af9b769897395aac526ca6744222fcf543787b16f58980844d6c49f65bbf0a62502d69646c58f14a8a29ba60d43c85a2a5ef1ed103c9a0e18d6491e4b0bafd2c25e290028d19203d4366ae5651aafd48abaf485588fd6a25fbe23c2c9a644a24337a041966b09cbb73dc3a080bf79564d15a26e97c1bff4f345ee256a9" />
   </ItemGroup>