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

Replace deprecated string:tokens/2 and string:strip/3 #38

Closed
wants to merge 0 commits into from
Closed

Replace deprecated string:tokens/2 and string:strip/3 #38

wants to merge 0 commits into from

Conversation

benbro
Copy link
Contributor

@benbro benbro commented Feb 22, 2018

Replace deprecated string:tokens/2 and string:strip/3 with string:lexemes/2 and string:trim/3.

@lrascao
Copy link
Owner

lrascao commented Feb 22, 2018

it's failing on all other otp versions besides 20, we need to come up with a portable way of performing the same functionality

@ferd
Copy link

ferd commented Feb 22, 2018

You might end up needing to take the approach we've taken for relx, rebar3, and erlware_commons and use optional compilation with a shim module or wrapper function.

@lrascao
Copy link
Owner

lrascao commented Feb 22, 2018

hope not, but yeah, it's an option

@benbro
Copy link
Contributor Author

benbro commented Feb 22, 2018

The old string functions will probably be supported.
Is it possible to keep using them and not fail on warnings?

@lrascao
Copy link
Owner

lrascao commented Feb 22, 2018

i'll take a look see first if there's another way of doing the same thing before dropping warnings_as_errors

@lrascao lrascao closed this Feb 24, 2018
@lrascao
Copy link
Owner

lrascao commented Feb 24, 2018

sorry, didn't mean to close this

@lrascao
Copy link
Owner

lrascao commented Feb 24, 2018

@benbro are you able to re-open it? i can't for some reason

@benbro
Copy link
Contributor Author

benbro commented Feb 24, 2018

I can't. The commit disappeared from my branch.

@lrascao
Copy link
Owner

lrascao commented Feb 24, 2018

damn, my bad, i fucked up with a force push, i'll open a new one with your changes and mine

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

Successfully merging this pull request may close these issues.

None yet

3 participants