From ae1e5bf11020df63a906e97eaa505432cfe075a7 Mon Sep 17 00:00:00 2001
From: Romeo Dumitrescu <romeo@drc.ro>
Date: Sat, 15 Jun 2024 17:56:21 +0300
Subject: [PATCH] Refactored build properties

- Removed a redundant PropertyGroup
- Moved the NoWarn property for NU5104 warning to the ES.FX projects' PropertyGroup
- Reorganized comments for better readability and understanding of code
---
 Directory.Build.props | 9 +++------
 1 file changed, 3 insertions(+), 6 deletions(-)

diff --git a/Directory.Build.props b/Directory.Build.props
index 7d08e96..0074a85 100644
--- a/Directory.Build.props
+++ b/Directory.Build.props
@@ -17,20 +17,17 @@
 		<PackageLicenseExpression>MIT</PackageLicenseExpression>
 		<!--<PackageOutputPath>$(PackageOutputDir)</PackageOutputPath>-->
 	</PropertyGroup>
-	
-	<PropertyGroup>
-		<!-- Don't warn about "A stable release of a package should not have a prerelease dependency." -->
-		<NoWarn>$(NoWarn);NU5104</NoWarn>
-	</PropertyGroup>
 
 	<PropertyGroup>
 		<!-- Default to false; specific projects will override to true -->
 		<GeneratePackageOnBuild>false</GeneratePackageOnBuild>
 	</PropertyGroup>
 
-	<!-- ES.FX Projects should build packages -->
 	<PropertyGroup Condition="'$(MSBuildProjectName)' == '' Or $(MSBuildProjectName.StartsWith('ES.FX'))">
+		<!-- ES.FX Projects should build packages -->
 		<GeneratePackageOnBuild>true</GeneratePackageOnBuild>
+		<!-- Don't warn about "A stable release of a package should not have a prerelease dependency." -->
+		<NoWarn>$(NoWarn);NU5104</NoWarn>
 	</PropertyGroup>
 
 	<!--Test projects-->