Build settings: analyzers on, deterministic builds, warnings-as-errors locally only

Author: FirdevsTorlak

Directory.Build.props

@@ -1,12 +1,29 @@
 <Project>
+  <!-- Global build settings for all projects in this repository -->
   <PropertyGroup>
+    <!-- Modern C# and safer null handling -->
     <LangVersion>latest</LangVersion>
     <Nullable>enable</Nullable>
-    <TreatWarningsAsErrors>true</TreatWarningsAsErrors>
+
+    <!-- Deterministic builds help reproducibility (CI/artifacts) -->
+    <Deterministic>true</Deterministic>
+
+    <!-- Roslyn analyzers on with a broad ruleset -->
     <EnableNETAnalyzers>true</EnableNETAnalyzers>
     <AnalysisMode>AllEnabledByDefault</AnalysisMode>
-    <Deterministic>true</Deterministic>
-    <!-- CI-friendly builds -->
-    <ContinuousIntegrationBuild Condition="'$(CI)' == 'true'">true</ContinuousIntegrationBuild>
+
+    <!-- Be strict locally, more tolerant in CI:
+         - On developer machines: warnings fail the build.
+         - In CI: let warnings be warnings (prevents unexpected red builds). -->
+    <TreatWarningsAsErrors Condition="'$(CI)' == '' and '$(GITHUB_ACTIONS)' == ''">true</TreatWarningsAsErrors>
+
+    <!-- Nicer error messages in logs -->
+    <GenerateFullPaths>true</GenerateFullPaths>
+  </PropertyGroup>
+
+  <!-- Release-specific knobs (optional but common) -->
+  <PropertyGroup Condition="'$(Configuration)' == 'Release'">
+    <Optimize>true</Optimize>
+    <CheckForOverflowUnderflow>false</CheckForOverflowUnderflow>
   </PropertyGroup>
 </Project>