-
Notifications
You must be signed in to change notification settings - Fork 2.2k
Fix/issue 4578 path normalization for unix and windows #4614
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
base: main
Are you sure you want to change the base?
Fix/issue 4578 path normalization for unix and windows #4614
Conversation
|
@kashifkhan0771 please review this PR |
|
Approved from my side, but these changes also require approval from @trufflesecurity/product-eng. Unfortunately, most of the team is currently out on holidays, so we’ll need to wait a bit. Thanks for the PR - once the team is back, we’ll get it merged 🙂 |
|
@kashifkhan0771 sorry to ping you can you @ mention rufflesecurity/product-eng team to review the PR :) |
rosecodym
left a comment
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.
Thanks for this! It seems like a great candidate for some new automated tests - would you mind adding some?
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!
Thank you for the contribution. Please add some tests, as suggested by Cody.
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.
I reviewed without looking at the other reviews, I third the request for tests haha
camgunz
left a comment
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.
LG, thanks!
|
@rosecodym please review & merge this one if it looks good to you. |
The issue described in below ticket was regenerated and fixed in this PR.
#4578
Resolves issues with cloning local Git repositories on Windows: