Make sure we do not upload artifacts during a pull request (PowerShel…

…l#7535)
webmote · Aug 16, 2018 · 0a1a47e · 0a1a47e
1 parent 4acf103
commit 0a1a47e
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 3 deletions.
diff --git a/build.psm1 b/build.psm1
@@ -1331,7 +1331,11 @@ function Publish-TestResults
     {
         $resolvedPath = (Resolve-Path -Path $Path).ProviderPath
         Write-Host "##vso[results.publish type=$Type;mergeResults=true;runTitle=$Title;publishRunAttachments=true;resultFiles=$resolvedPath;]"
-        Write-Host "##vso[artifact.upload containerfolder=testResults;artifactname=testResults]$resolvedPath"
+
+        if($env:BUILD_REASON -ne 'PullRequest')
+        {
+            Write-Host "##vso[artifact.upload containerfolder=testResults;artifactname=testResults]$resolvedPath"
+        }
     }
 }
 

diff --git a/test/tools/Modules/HelpersCommon/HelpersCommon.psm1 b/test/tools/Modules/HelpersCommon/HelpersCommon.psm1
@@ -233,8 +233,11 @@ function Send-VstsLogFile {
         Copy-Item -Path $Path -Destination $logFile
     }
 
-    Write-Host "##vso[artifact.upload containerfolder=$name;artifactname=$name]$logFile"
-    Write-Verbose "Log file captured as $name" -Verbose
+    if($env:BUILD_REASON -ne 'PullRequest')
+    {
+        Write-Host "##vso[artifact.upload containerfolder=$name;artifactname=$name]$logFile"
+        Write-Verbose "Log file captured as $name" -Verbose
+    }
 }
 
 # Tests if the Linux or macOS user is root