-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Fix slnf test assets to use \\
#51791
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
Fix slnf test assets to use \\
#51791
Conversation
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.
Pull Request Overview
This PR fixes the path separator convention in solution filter (.slnf) test assets to use double backslashes (\\) instead of forward slashes. The change aligns these test assets with the JSON format standard and the existing convention used in all other .slnf files in the repository, where backslashes must be escaped in JSON strings.
Key Changes:
- Updated path separators from
/to\\in .slnf test asset files to match the repository-wide convention
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| test/TestAssets/TestProjects/MultiTestProjectSolutionWithTests/TestProjects.slnf | Changed project path separator from / to \\ |
| test/TestAssets/TestProjects/MultiTestProjectSolutionWithTests/SolutionFilter/OtherTestProjects.slnf | Changed both solution path and project path separators from / to \\ |
57fe217 to
f3e246b
Compare
Let's see after #51411. |
f3e246b to
9b50da6
Compare
Closes #51521
I think the PR might fail currently, but hopefully will pass after #51411 is merged.