From 768cad16718650f120dd42acfcc9cf5c4dec555f Mon Sep 17 00:00:00 2001 From: Jonathan Dodds Date: Mon, 1 May 2023 16:40:40 -0400 Subject: [PATCH] Code cleanup in Microsoft.Common.tasks (#8657) Context Minor code cleanup in Microsoft.Common.tasks. Changes Made Removed duplicate UsingTask for the ResolveSDKReference task. The redundant ResolveSDKReference doesn't seem to create harm but is not useful. Alphabetized the UsingTask elements. Retained the separate grouping of Roslyn tasks. Retained the blank lines around tasks that have different Runtimes and/or Conditions (i.e. GenerateResource, RegisterAssembly, and UnregisterAssembly). Ordering the UsingTask elements is intended to aid inspection and maintenance of tasks. Testing Tested on Windows 11 and macOS 12. Tested by running unit tests and by having this change in several development branches where msbuild has been run on project files. Notes This change is included in the implementation for #8613, which is PR #8614. --- src/Tasks/Microsoft.Common.tasks | 18 ++++++++---------- 1 file changed, 8 insertions(+), 10 deletions(-) diff --git a/src/Tasks/Microsoft.Common.tasks b/src/Tasks/Microsoft.Common.tasks index d91e88e9861..beab1878e60 100644 --- a/src/Tasks/Microsoft.Common.tasks +++ b/src/Tasks/Microsoft.Common.tasks @@ -16,9 +16,9 @@ - + @@ -32,7 +32,6 @@ - @@ -53,8 +52,9 @@ + - + @@ -72,15 +72,13 @@ - - - - + + @@ -90,21 +88,21 @@ - + + - + -