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

Stop monkey-patching String with #snakecase #102

Merged
merged 1 commit into from
Mar 1, 2024
Merged

Stop monkey-patching String with #snakecase #102

merged 1 commit into from
Mar 1, 2024

Conversation

mchu
Copy link
Contributor

@mchu mchu commented Feb 29, 2024

What kind of change is this?
Fixes #101

Stops the monkey patching of #snakecase onto String.

Did you add tests for your changes?
Updated existing String#snakecase tests to test Nori::StringUtils.snakecase.

Summary of changes
Moved String#snakecase into a module class method.

Other information
Related to savonrb/savon#984

@pcai pcai merged commit dbbd948 into main Mar 1, 2024
12 checks passed
@pcai pcai deleted the bugfix/101 branch March 1, 2024 00:45
@choosen
Copy link

choosen commented Sep 5, 2024

Savon gave it breaking change, not monkeyPatch

BC BREAKING Fix: #988 Savon no longer monkeypatches String#snakecase

You changed the util so it's hard to be compatible with Nori when somebody used pointed out in readme before 'StringKey'.snakecase

It's of course good to not pollute String class, but it should be minor bump I suppose instead of bugfix

@pcai
Copy link
Member

pcai commented Sep 5, 2024

Yes in hindsight it would’ve been a point release as with Savon - sorry about that

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

Successfully merging this pull request may close these issues.

dont monkeypatch String#snakecase
3 participants