Skip to content
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 all docpaths to unix format paths #1442

Merged
merged 1 commit into from
Jan 14, 2022

Conversation

Wraith2
Copy link
Contributor

@Wraith2 Wraith2 commented Dec 22, 2021

A simple find-replace on windows style paths to unix style paths. This will make it easier to see code differences in the large files that need merging like SqlDataReader.

@DavoudEshtehari this contributes to merge,

@DavoudEshtehari DavoudEshtehari added the ➕ Code Health Issues/PRs that are targeted to source code quality improvements. label Dec 22, 2021
@DavoudEshtehari DavoudEshtehari added this to the 5.0.0-preview1 milestone Dec 22, 2021
@Wraith2
Copy link
Contributor Author

Wraith2 commented Jan 14, 2022

This is the result of find and replace on a very specific doc string and the library won't compile if the paths are wrong. Given that the CI is green and this cannot cause any behavioural changes just how much reviewing does it need?

@johnnypham
Copy link
Contributor

This is the result of find and replace on a very specific doc string and the library won't compile if the paths are wrong. Given that the CI is green and this cannot cause any behavioural changes just how much reviewing does it need?

I had to count every single slash and period. Jk, approved.

@johnnypham johnnypham merged commit 2f50297 into dotnet:main Jan 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
➕ Code Health Issues/PRs that are targeted to source code quality improvements.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants