Merge pull request #2291 from svengeance/feature/WebSdkInProjectFiles

Allows usage of Web SDK when updating project files

Author: asbjornu

src/GitVersionCore.Tests/VersionConverters/ProjectFileUpdaterTests.cs

@@ -52,6 +52,25 @@ public void CanUpdateProjectFileWithStandardProjectFileXml(string xml)
             canUpdate.ShouldBe(true);
         }
 
+        [TestCase(@"
+<Project Sdk=""Microsoft.NET.Sdk.Web"">
+  <PropertyGroup>
+    <OutputType>Exe</OutputType>
+    <TargetFramework>netcoreapp3.1</TargetFramework>
+  </PropertyGroup>
+</Project>
+")]
+        [Category(NoMono)]
+        [Description(NoMonoDescription)]
+        public void CanUpdateProjectFileWithStandardWebProjectFileXml(string xml)
+        {
+            using var projectFileUpdater = new ProjectFileUpdater(log, fileSystem);
+
+            var canUpdate = projectFileUpdater.CanUpdateProjectFile(XElement.Parse(xml));
+
+            canUpdate.ShouldBe(true);
+        }
+
         [TestCase(@"
 <Project Sdk=""SomeOtherProject.Sdk"">
   <PropertyGroup>

src/GitVersionCore/VersionConverters/AssemblyInfo/ProjectFileUpdater.cs

@@ -102,11 +102,12 @@ internal bool CanUpdateProjectFile(XElement xmlRoot)
                 return false;
             }
 
-            var supportedSdk = "Microsoft.NET.Sdk";
+            var supportedSdks = new[] { "Microsoft.NET.Sdk", "Microsoft.NET.Sdk.Web" };
             var sdkAttribute = xmlRoot.Attribute("Sdk");
-            if (sdkAttribute == null || sdkAttribute.Value != supportedSdk)
+            if (sdkAttribute == null || !supportedSdks.Contains(sdkAttribute.Value))
             {
-                log.Warning($"Specified project file Sdk ({sdkAttribute?.Value}) is not supported, please ensure the project sdk is {supportedSdk}.");
+                var supportedSdkString = string.Join("|", supportedSdks);
+                log.Warning($"Specified project file Sdk ({sdkAttribute?.Value}) is not supported, please ensure the project sdk is of the following: {supportedSdkString}.");
                 return false;
             }