-
Notifications
You must be signed in to change notification settings - Fork 730
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
[E2E] Modify commands to address running on Windows. #13682
Conversation
@intel/llvm-reviewers-runtime, a friendly request to have this reviewed please. |
@mmoadeli Why are we dropping the last test where we run the test without the envvar and check that the dot files aren't there? Is this intended? |
thank you @maarquitos14, that is put back. |
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.
LGTM.
Running the test on Windows failed due to missing support of `ls`. Replacing `ls` with `cat` made the test pass on Windows.
Running the test on Windows failed due to missing support of
ls
. Replacingls
withcat
made the test pass on Windows.contributor @ldrumm