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

Change Pascalize to remove spaces fix #774 #793

Merged
merged 6 commits into from
Mar 11, 2019

Conversation

AKTheKnight
Copy link
Contributor

This is a PR to change to fix #774 that changes Pascalize (and therefore Camelize too) to treat spaces in the same way it treats underscores.

I wasn't sure whether to have this as an optional change, or a change to the method itself, so I've chosen to just change the method and create this PR as a way to generate conversation around the issue.

Let me know if you have any thoughts.

@clairernovotny
Copy link
Member

Overall this change is fine, but something seems to have gotten messed up with the readme diff?

@AKTheKnight
Copy link
Contributor Author

Not sure what happened there, think I've fixed it and updated the branch

@AKTheKnight
Copy link
Contributor Author

The latest commits remove the second regex line (it wasn't actually needed), and also add support for multiple spaces in a row.

Not sure what the thoughts are on adding support for multiple underscores in a row?

@clairernovotny clairernovotny merged commit e29de81 into Humanizr:master Mar 11, 2019
@AKTheKnight AKTheKnight deleted the fix-774 branch March 16, 2019 11:14
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.

Pascalize and Camelize do not join words separated by spaces
2 participants