Skip to content

ENH: StringMethods now supports ljust and rjust #9352

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

Merged
merged 1 commit into from
Jan 29, 2015

Conversation

sinhrks
Copy link
Member

@sinhrks sinhrks commented Jan 25, 2015

Derived from #9111.

@jreback jreback added the Strings String extension data type and string data label Jan 25, 2015
@jreback jreback added this to the 0.16.0 milestone Jan 25, 2015
@jreback
Copy link
Contributor

jreback commented Jan 25, 2015

this needs to go after #9322 (and might need a rebase after that) as the docs are changing a bit)

@shoyer
Copy link
Member

shoyer commented Jan 25, 2015

Just merged #9322, so this needs a rebase now.

@@ -576,7 +580,7 @@ def str_findall(arr, pat, flags=0):
return _na_map(regex.findall, arr)


def str_pad(arr, width, side='left'):
def str_pad(arr, width, side='left', fillchar=' '):
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 here an explanation of the new kwarg fillchar as well?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes, added.

jreback added a commit that referenced this pull request Jan 29, 2015
ENH: StringMethods now supports ljust and rjust
@jreback jreback merged commit 0b789eb into pandas-dev:master Jan 29, 2015
@jreback
Copy link
Contributor

jreback commented Jan 29, 2015

thanks @sinhrks

@sinhrks sinhrks deleted the string_fillchar branch January 29, 2015 23:28
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.

4 participants