-
Notifications
You must be signed in to change notification settings - Fork 29.1k
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
url: modify pathToFileURL to handle extended UNC paths #54262
Conversation
Review requested:
|
2645ec4
to
822a4ac
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #54262 +/- ##
=======================================
Coverage 87.10% 87.10%
=======================================
Files 647 647
Lines 181754 181759 +5
Branches 34883 34887 +4
=======================================
+ Hits 158316 158323 +7
- Misses 16741 16748 +7
+ Partials 6697 6688 -9
|
822a4ac
to
92e004f
Compare
92e004f
to
bb8a0ce
Compare
bb8a0ce
to
68f519f
Compare
It seems that the failed test |
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
Landed in 4dc1ae0 |
PR-URL: #54262 Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: James M Snell <[email protected]>
PR-URL: #54262 Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: James M Snell <[email protected]>
Fixes #54261