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

Unify lpad, rpad and cpad and move to external package #22241

Closed
dpsanders opened this issue Jun 6, 2017 · 4 comments
Closed

Unify lpad, rpad and cpad and move to external package #22241

dpsanders opened this issue Jun 6, 2017 · 4 comments
Labels
strings "Strings!"

Comments

@dpsanders
Copy link
Contributor

lpad, rpad and cpad would seem to be good candidates to be unified in a pad function.

There are a lot of such string manipulation functions that feel like they should be moved from Base to a string manipulation package.

@KristofferC
Copy link
Member

I don't see any benefit in moving short, pure, julia code that is generally useful to a package.

@quinnj
Copy link
Member

quinnj commented Jun 6, 2017

I could see us making an official Strings module that would still be apart of Base/StdLib, but yeah, a lot of these functions are usually expected to be in the "default" language.

@StefanKarpinski
Copy link
Member

We could always just use https://api.left-pad.io/

@kshyatt kshyatt added the strings "Strings!" label Jun 21, 2017
@KristofferC
Copy link
Member

I don't think this will be moved out of this repo. Perhaps to a String stdlib as part of #25416

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
strings "Strings!"
Projects
None yet
Development

No branches or pull requests

5 participants