Merge pull request #2 from nblumhardt/dev

Define DEBUG so that Debug calls aren't compiled out

Author: nblumhardt

src/Serilog.Sinks.Debug/DebugLoggerConfigurationExtensions.cs

@@ -38,7 +38,7 @@ public static class ConsoleLoggerConfigurationExtensions
         /// <param name="levelSwitch">A switch allowing the pass-through minimum level
         /// to be changed at runtime.</param>
         /// <param name="outputTemplate">A message template describing the format used to write to the sink.
-        /// the default is <code>"[{Timestamp:HH:mm:ss} {Level:u3}] {Message}{NewLine}{Exception}"</code>.</param>
+        /// the default is <code>"[{Timestamp:HH:mm:ss} {Level:u3}] {Message:lj}{NewLine}{Exception}"</code>.</param>
         /// <param name="formatProvider">Supplies culture-specific formatting information, or null.</param>
         /// <returns>Configuration object allowing method chaining.</returns>
         public static LoggerConfiguration Debug(

src/Serilog.Sinks.Debug/Serilog.Sinks.Debug.csproj

@@ -24,6 +24,11 @@
     <RootNamespace>Serilog</RootNamespace>
   </PropertyGroup>
 
+  <PropertyGroup>
+    <!-- This is required so that calls to System.Diagnostics.Debug aren't compiled out of the release binary. -->
+    <DefineConstants>$(DefineConstants);DEBUG</DefineConstants>
+  </PropertyGroup>
+
   <ItemGroup>
     <PackageReference Include="Serilog" Version="2.5.0" />
     <PackageReference Include="System.Diagnostics.Debug" Version="4.3.0" />

test/Serilog.Sinks.Debug.Tests/DebugSinkTests.cs

@@ -0,0 +1,7 @@
+namespace Serilog.Sinks.Debug.Tests
+{
+    public class DebugSinkTests
+    {
+        // To do, need some tests here...
+    }
+}

test/Serilog.Sinks.Debug.Tests/Serilog.Sinks.Debug.Tests.csproj

@@ -28,4 +28,9 @@
     <Service Include="{82a7f48d-3b50-4b1e-b82e-3ada8210c358}" />
   </ItemGroup>
 
+  <PropertyGroup>
+    <!-- This is required so that calls to System.Diagnostics.Debug aren't compiled out of the release binary. -->
+    <DefineConstants>$(DefineConstants);DEBUG</DefineConstants>
+  </PropertyGroup>
+
 </Project>