-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
[stdlib] Add os.path.splitroot
#3780
Conversation
Signed-off-by: Mikhail Tavarez <[email protected]>
a352080
to
6d54656
Compare
Signed-off-by: Mikhail Tavarez <[email protected]>
@bgreni After I finished this, I noticed you had splitroot in an earlier PR that hasn't been merged. Hope you don't mind that I borrowed one of your test cases! |
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, thanks!
!sync |
✅🟣 This contribution has been merged 🟣✅ Your pull request has been merged to the internal upstream Mojo sources. It will be reflected here in the Mojo repository on the nightly branch during the next Mojo nightly release, typically within the next 24-48 hours. We use Copybara to merge external contributions, click here to learn more. |
[External] [stdlib] Add `os.path.splitroot` Also cleaned up a previous changelog entry so it aligns with the function's intent. Co-authored-by: Mikhail Tavarez <[email protected]> Closes #3780 MODULAR_ORIG_COMMIT_REV_ID: 081959627df8f25d7f6e04856ca03ab992ecca6c
Landed in ced5633! Thank you for your contribution 🎉 |
Also cleaned up a previous changelog entry so it aligns with the function's intent.