From 4b1e0d09c74d0fbc8460783d0cb107d726410043 Mon Sep 17 00:00:00 2001
From: Nate McMaster <nate.mcmaster@microsoft.com>
Date: Fri, 17 Aug 2018 11:34:22 -0700
Subject: [PATCH] Skip npm package publishing if there are no npm packages to
 release

---
 build/Publish.targets | 11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)

diff --git a/build/Publish.targets b/build/Publish.targets
index 2c97b996eb..d5f2c74fc3 100644
--- a/build/Publish.targets
+++ b/build/Publish.targets
@@ -209,11 +209,13 @@
       <AuthTokenSetting>$(PublishMyGetNpmRegistryUrl.Replace("https:", "")):_authToken</AuthTokenSetting>
     </PropertyGroup>
 
-    <Exec Command="npm config set &quot;$(AuthTokenSetting)&quot; $(PublishMyGetFeedKey)"
+    <Exec Condition=" '$(ContainsNpmPackageToPublish)' == 'true' "
+          Command="npm config set &quot;$(AuthTokenSetting)&quot; $(PublishMyGetFeedKey)"
           StandardOutputImportance="Normal" />
 
     <!-- When you UseCommandProcessor FileName is ignored -->
-    <Run FileName="cmd"
+    <Run Condition=" @(PackageToPublish->Count()) != 0 "
+         FileName="cmd"
          Arguments="npm;publish;--registry;$(PublishMyGetNpmRegistryUrl);%(NpmPackageToPublish.Identity)"
          MaxRetries="5"
          UseCommandProcessor="true"
@@ -221,10 +223,11 @@
       <Output TaskParameter="ExitCode" ItemName="_NpmExitCodes" />
     </Run>
 
-    <Exec Command="npm config delete $(AuthTokenSetting)"
+    <Exec Condition=" @(PackageToPublish->Count()) != 0 "
+          Command="npm config delete $(AuthTokenSetting)"
           StandardOutputImportance="Normal" />
 
-    <Error Text="Publishing npm modules failed" Condition="%(_NpmExitCodes.Identity) != 0" />
+    <Error Text="Publishing npm modules failed" Condition=" @(PackageToPublish->Count()) != 0 AND %(_NpmExitCodes.Identity) != 0" />
   </Target>
 
   <Target Name="PublishToAzureFeed"