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
DM-38552: Support non-file root in constructor for relative paths #47
Conversation
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #47 +/- ##
==========================================
+ Coverage 84.47% 85.49% +1.01%
==========================================
Files 27 27
Lines 3665 3633 -32
Branches 767 746 -21
==========================================
+ Hits 3096 3106 +10
+ Misses 441 416 -25
+ Partials 128 111 -17
... and 11 files with indirect coverage changes Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report in Codecov by Sentry. |
Previously we tried to distinguish between someone constructing a ResourcePath with an schemeless absolute path and someone constructing one with forceAbsolute=True and the distinction was mostly confusing to everyone.
eg TYPE_CHECKING blocks.
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.
Looks good, few minor questions.
Also fix the strange behavior where absolute schemeless URIs can sometimes be updated to
file
URIs but not always.Checklist
doc/changes