-
-
Notifications
You must be signed in to change notification settings - Fork 18.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
ENH: added rsplit to StringMethods #10303
Conversation
[u('f'), u('g'), u('h')]]) | ||
tm.assert_series_equal(result, exp) | ||
|
||
def test_rsplit(self): |
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.
Can you add some cases using maxsplit
and expand=True
?
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.
it's added
Thanks! I think this is OK for 0.16.2 as a follow-up of string enh if there is no objections. Could you update:
|
acdf6ac
to
b267437
Compare
@sinhrks it's updated, please take a look |
@@ -79,6 +79,8 @@ See the :ref:`documentation <basics.pipe>` for more. (:issue:`10129`) | |||
.. _magrittr: https://github.com/smbache/magrittr | |||
.. _R: http://www.r-project.org | |||
|
|||
- Added `rsplit` to Index/Series StringMethods |
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.
add the PR number here as the issue number
ENH: added rsplit to StringMethods
thanks! |
as a part of #9111
cc @sinhrks