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.

Sign up for GitHub

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

Jump to bottom

Proper fix for stringcase and add unit test #36

Open

andy31415 wants to merge 1 commit into okunishinishi:master from andy31415:master

andy31415 commented May 4, 2023

This fixes the breakage in #23 and adds a unit tests and fixes the case for the spacing that the original attempt for fixing addressed.


          Proper fix for stringcase and add unit test

dc9843f

Author

andy31415 commented May 4, 2023

Overall I am unsure about stringcase and pascalcase test coverage.
current pip install does things like:

>>> import stringcase
>>> stringcase.camelcase('THIS_IS_A_TEST')
'tHIS_IS_A_TEST'
>>> stringcase.camelcase('With Some Spaces')
'witompaces'
>>>

Which seem somewhat unexpected. Ideally I would get thisIsATest or withSomeSpaces .... seems we may go down a rabbit hole here.

bkmgit mentioned this pull request

better implementation of camelcase #31

Open

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

Labels

None yet