Skip to content
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

Merged
merged 1 commit into from
Jun 9, 2015
Merged

Conversation

mortada
Copy link
Contributor

@mortada mortada commented Jun 6, 2015

as a part of #9111

cc @sinhrks

@sinhrks sinhrks added API Design Strings String extension data type and string data labels Jun 6, 2015
@sinhrks sinhrks added this to the 0.16.2 milestone Jun 6, 2015
[u('f'), u('g'), u('h')]])
tm.assert_series_equal(result, exp)

def test_rsplit(self):
Copy link
Member

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?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it's added

@sinhrks
Copy link
Member

sinhrks commented Jun 6, 2015

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:

  • Release note
  • text.rst and api.rst

@mortada mortada force-pushed the str_rsplit branch 2 times, most recently from acdf6ac to b267437 Compare June 8, 2015 06:23
@mortada
Copy link
Contributor Author

mortada commented Jun 8, 2015

@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
Copy link
Contributor

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

jreback added a commit that referenced this pull request Jun 9, 2015
ENH: added rsplit to StringMethods
@jreback jreback merged commit b206548 into pandas-dev:master Jun 9, 2015
@jreback
Copy link
Contributor

jreback commented Jun 9, 2015

thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API Design Strings String extension data type and string data
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants