Unlink dependencies if tests fail (#6483)

- Results are published for both success and failure - Unlink dependencies should use the same condition as publish - Continuation of #6418
Azure · Dec 10, 2019 · a46af7a · a46af7a
1 parent bccb886
commit a46af7a
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 0 deletions.
diff --git a/eng/pipelines/templates/jobs/archetype-sdk-client.yml b/eng/pipelines/templates/jobs/archetype-sdk-client.yml
@@ -257,6 +257,7 @@ jobs:
         # which need to walk the directory tree (and are hardcoded to follow symlinks).
         - script: |
             node eng/tools/rush-runner.js unlink
+          condition: succeededOrFailed()
           displayName: "Unlink dependencies"
 
         # It's important for performance to pass "sdk" as "searchFolder" to avoid looking under root "node_modules".

diff --git a/eng/pipelines/templates/jobs/archetype-sdk-integration.yml b/eng/pipelines/templates/jobs/archetype-sdk-integration.yml
@@ -134,6 +134,7 @@ jobs:
       # which need to walk the directory tree (and are hardcoded to follow symlinks).
       - script: |
           node eng/tools/rush-runner.js unlink
+        condition: succeededOrFailed()
         displayName: "Unlink dependencies"
 
       # It's important for performance to pass "sdk" as "searchFolder" to avoid looking under root "node_modules".