From 10b53ccf20d6cc37815334aa63c990befbbc0767 Mon Sep 17 00:00:00 2001
From: ta264 <ta264@users.noreply.github.com>
Date: Mon, 30 Sep 2019 21:33:03 +0100
Subject: [PATCH] New: Treat compile warnings as errors

---
 src/Directory.Build.props                             | 3 +++
 src/NzbDrone.Core/Notifications/Email/EmailService.cs | 3 +++
 2 files changed, 6 insertions(+)

diff --git a/src/Directory.Build.props b/src/Directory.Build.props
index b3d5c08be..6b93596a0 100644
--- a/src/Directory.Build.props
+++ b/src/Directory.Build.props
@@ -1,6 +1,8 @@
 <Project>
   <!-- Common to all Lidarr Projects -->
   <PropertyGroup>
+    <TreatWarningsAsErrors>true</TreatWarningsAsErrors>
+
     <LidarrRootDir>$(MSBuildThisFileDirectory)..\</LidarrRootDir>
 
     <!-- Specifies the type of output -->
@@ -63,6 +65,7 @@
     
     <Deterministic Condition="$(AssemblyVersion.EndsWith('*'))">False</Deterministic>
   </PropertyGroup>
+
   <!-- Set the AssemblyConfiguration attribute for projects -->
   <ItemGroup Condition="'$(LidarrProject)'=='true'">
     <AssemblyAttribute Include="System.Reflection.AssemblyConfigurationAttribute">
diff --git a/src/NzbDrone.Core/Notifications/Email/EmailService.cs b/src/NzbDrone.Core/Notifications/Email/EmailService.cs
index 7cd8bf0c5..68b11043c 100644
--- a/src/NzbDrone.Core/Notifications/Email/EmailService.cs
+++ b/src/NzbDrone.Core/Notifications/Email/EmailService.cs
@@ -51,7 +51,10 @@ namespace NzbDrone.Core.Notifications.Email
 
         private void Send(MailMessage email, string server, int port, bool ssl, NetworkCredential credentials)
         {
+            //TODO: This is marked obsolete in mono, will not be necessary once we build with net core instead
+#pragma warning disable CS0618
             var smtp = new SmtpClient(server, port);
+#pragma warning restore CS0618
             smtp.EnableSsl = ssl;
             smtp.Credentials = credentials;