-
Notifications
You must be signed in to change notification settings - Fork 10.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add clean.ps1 and clean.sh #11154
Add clean.ps1 and clean.sh #11154
Conversation
dougbu
commented
Jun 12, 2019
- Add a clean.cmd command #10463
clean.sh
Outdated
|
||
# This incantation avoids unbound variable issues if git_args is empty | ||
# https://stackoverflow.com/questions/7577052/bash-empty-array-expansion-with-set-u | ||
git clean -dix -e .dotnet/ -e .tools/ -e src/SignalR/clients/ts/FunctionalTests/node_modules/ ${git_args[@]+"${git_args[@]}"} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
git clean -dix -e .dotnet/ -e .tools/ -e src/SignalR/clients/ts/FunctionalTests/node_modules/ ${git_args[@]+"${git_args[@]}"} | |
git clean -dix -e .dotnet/ -e .tools/ ${git_args[@]+"${git_args[@]}"} |
Git on non-Windows handles symlinks correctly, so we don't need this exclusion :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems fine to me. Would be good to get @Tratcher's approval -- he was one of the devs requesting this.
That looks helpful. How about cmd? We have cmd for most other common commands. |
Took your suggestion @Tratcher |
Another now-standard timeout of the build. Merging since issue can't be related to these three new files. |