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

Rename string trim functions #315

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

cmeeren
Copy link
Contributor

@cmeeren cmeeren commented Apr 25, 2020

As requested in #288 (comment)

@wallymathieu
Copy link
Member

Looks good. Looks like trimboth is used in python. Another alternative is to use the word strip.

@wallymathieu
Copy link
Member

It would be good to hear some people chime in here 😃

@cmeeren
Copy link
Contributor Author

cmeeren commented Apr 25, 2020

I think we should stick to some version of trim because it's what .NET users expect.

Some alternatives to trimBoth are trimStartEnd and trimChars (though in the latter case it's not consistent with trimStart and trimEnd).

@wallymathieu
Copy link
Member

That's true. Let's see what people says 😃

@gusty
Copy link
Member

gusty commented Apr 27, 2020

I think I would be in favor of this change, but of course in case we approve it, it will have to be for v2.0

In the meantime it would be interesting to hear other voices. Maybe @cannorin can comment as he added many string functions to the lib.

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

Successfully merging this pull request may close these issues.

None yet

4 participants