From 7cbe23f94d037deb1082a57508c19007f4634bd1 Mon Sep 17 00:00:00 2001
From: Joe DeCock <josephdecock@gmail.com>
Date: Wed, 5 Mar 2025 14:21:13 -0600
Subject: [PATCH] Make debug builds less strict

Release builds do more work and are stricter. This allows for faster
development: devs can be messy/experimental initially and verify their
code easily by running a build in Release mode, and we also enforce
stricter rules in CI with the Release configuration. We can also move
any slow actions to the Release configuration to speed up test runs, etc.
---
 samples.props | 12 +++++++++++-
 src.props     | 15 +++++++++++++--
 test.props    | 10 ++++++++++
 3 files changed, 34 insertions(+), 3 deletions(-)

diff --git a/samples.props b/samples.props
index 508c95bc6..414df8ac4 100644
--- a/samples.props
+++ b/samples.props
@@ -3,10 +3,20 @@
 
   <PropertyGroup>
     <ManagePackageVersionsCentrally>true</ManagePackageVersionsCentrally>
-    <TreatWarningsAsErrors>true</TreatWarningsAsErrors>
     <IsPackable>false</IsPackable>
     <WarnOnPackingNonPackableProject>false</WarnOnPackingNonPackableProject>
     <ImplicitUsings>true</ImplicitUsings>
   </PropertyGroup>
 
+  <!--Release builds do more work and are stricter. This allows for faster
+      development: devs can be messy/experimental initially and verify their
+      code easily by running a build in Release mode, and we also enforce
+      stricter rules in CI with the Release configuration. We can also move
+      any slow actions to the Release configuration to speed up test runs, etc.
+  -->
+  <PropertyGroup Condition="'$(Configuration)'=='Release'">
+    <EnforceCodeStyleInBuild>true</EnforceCodeStyleInBuild>
+    <TreatWarningsAsErrors>true</TreatWarningsAsErrors>
+  </PropertyGroup>
+
 </Project>
\ No newline at end of file
diff --git a/src.props b/src.props
index 04bdc5d55..7cb74cc81 100644
--- a/src.props
+++ b/src.props
@@ -16,9 +16,8 @@
     <!-- <Nullable>enable</Nullable> -->
     <LangVersion>latest</LangVersion>
     <ImplicitUsings>true</ImplicitUsings>
-    <GenerateDocumentationFile>true</GenerateDocumentationFile>
+    
     <ManagePackageVersionsCentrally>true</ManagePackageVersionsCentrally>
-    <TreatWarningsAsErrors>true</TreatWarningsAsErrors>
     <ContinuousIntegrationBuild Condition="'$(GITHUB_ACTIONS)' == 'true'">True</ContinuousIntegrationBuild>
     <Deterministic>true</Deterministic>
     <NoWarn>$(NoWarn);CS1591,NU1507</NoWarn>
@@ -48,6 +47,18 @@
     <PackageReadmePath>../../README.md</PackageReadmePath>
   </PropertyGroup>
 
+  <!--Release builds do more work and are stricter. This allows for faster
+      development: devs can be messy/experimental initially and verify their
+      code easily by running a build in Release mode, and we also enforce
+      stricter rules in CI with the Release configuration. We can also move
+      any slow actions to the Release configuration to speed up test runs, etc.
+  -->
+  <PropertyGroup Condition="'$(Configuration)'=='Release'">
+    <EnforceCodeStyleInBuild>true</EnforceCodeStyleInBuild>
+    <TreatWarningsAsErrors>true</TreatWarningsAsErrors>
+    <GenerateDocumentationFile>true</GenerateDocumentationFile>
+  </PropertyGroup>
+
   <ItemGroup>
     <PackageReference Include="Microsoft.SourceLink.GitHub">
       <PrivateAssets>all</PrivateAssets>
diff --git a/test.props b/test.props
index 243a95dd3..ae1439cbd 100644
--- a/test.props
+++ b/test.props
@@ -12,6 +12,16 @@
     <NoWarn>$(NoWarn);NU1507</NoWarn>
   </PropertyGroup>
 
+  <!--Release builds do more work and are stricter. This allows for faster
+      development: devs can be messy/experimental initially and verify their
+      code easily by running a build in Release mode, and we also enforce
+      stricter rules in CI with the Release configuration. We can also move
+      any slow actions to the Release configuration to speed up test runs, etc.
+  -->
+  <PropertyGroup Condition="'$(Configuration)'=='Release'">
+    <EnforceCodeStyleInBuild>true</EnforceCodeStyleInBuild>
+  </PropertyGroup>
+
   <ItemGroup>
     <PackageReference Include="coverlet.collector">
       <PrivateAssets>all</PrivateAssets>