Adding support for running dotnet run after installing core tools thru npm (#3127)

* adding windows target

* updating release notes

* addressing comments

Author: aishwaryabh

sdk/Sdk/Targets/Microsoft.Azure.Functions.Worker.Sdk.targets

@@ -106,12 +106,19 @@ WARNING:  DO NOT MODIFY this file unless you are knowledgeable about MSBuild and
     <Error Condition="'$(_ToolingSuffix)' == ''" Text="Invalid combination of TargetFramework and AzureFunctionsVersion is set." />
   </Target>
 
-  <!-- This target is used to set up the run arguments for the function app, supporting `dotnet run` -->
   <Target Name="_FunctionsComputeRunArguments" BeforeTargets="ComputeRunArguments" DependsOnTargets="_FunctionsCheckForCoreTools">
-    <PropertyGroup Condition="'$(FuncExists)' == 'true'">
-      <RunCommand>func</RunCommand>
-      <RunArguments>host start $(RunArguments)</RunArguments>
-      <RunWorkingDirectory>$(OutDir)</RunWorkingDirectory>
+    <!-- Windows Configuration -->
+    <PropertyGroup Condition="'$(OS)' == 'Windows_NT'">
+        <RunCommand>cmd</RunCommand>
+        <RunArguments>/C func start $(RunArguments)</RunArguments>
+        <RunWorkingDirectory>$(OutDir)</RunWorkingDirectory>
+    </PropertyGroup>
+
+    <!-- Unix/Linux/macOS Configuration -->
+    <PropertyGroup Condition="'$(OS)' != 'Windows_NT'">
+        <RunCommand>func</RunCommand>
+        <RunArguments>start $(RunArguments)</RunArguments>
+        <RunWorkingDirectory>$(OutDir)</RunWorkingDirectory>
     </PropertyGroup>
   </Target>
 

sdk/release_notes.md

@@ -7,6 +7,7 @@
 ### Microsoft.Azure.Functions.Worker.Sdk <version>
 
 - Build no longer generates `functions.metadata` if source-generated metadata provider is enabled. (#2974)
+- Fixing `dotnet run` to work on Windows when core tools is installed from NPM (#3127)
 
 ### Microsoft.Azure.Functions.Worker.Sdk.Generators <version>