Skip to content

Commit

Permalink
Ignore errors from "rush purge" (#2430)
Browse files Browse the repository at this point in the history
- Workaround for microsoft/rushstack#4334
- Partially fixes Azure/typespec-azure#3473
  • Loading branch information
mikeharder authored Sep 16, 2023
1 parent fd2663c commit 8e30130
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions eng/pipelines/jobs/build-and-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -36,9 +36,10 @@ steps:

# Unlink node_modules folders to significantly improve performance of subsequent tasks
# which need to walk the directory tree (and are hardcoded to follow symlinks).
- script: node common/scripts/install-run-rush.js purge
# Append "|| cd ." to command to ignore errors (works on both Windows and Linux) as a workaround
# for microsoft/rushstack#4334.
- script: node common/scripts/install-run-rush.js purge || cd .
displayName: "Purge dependencies"
continueOnError: true # Some issue with deleting common/temp/rush-recycler folder on linux. This doesn't matter for the next steps.
condition: always()

# It's important for performance to pass "packages" as "searchFolder" to avoid looking under root "node_modules".
Expand Down

0 comments on commit 8e30130

Please sign in to comment.