-
Notifications
You must be signed in to change notification settings - Fork 48
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
digits problem. #31
Comments
As discussed in #22, it's non-trivial to do a perfect job of generically splitting strings in the presence of digits. My current recommendation is to use the
My long-term plan is to revise the library to be slighly more flexible.
So you'd like |
Hello, well I know this is a rather old issue but as I found it when trying to solve this precise problem, I thought I would suggest my solution. Basically, follow up
So, perhaps this avoids the need to make something complex inside CSK? Or perhaps it provides an idea for an internal fix? OK, it's not very clever or optimal (being a fix after the event) but it is nice and simple. |
I prefer numbers case should be recognised as a case of the preceding word also. Here we did a dirty alter-var-root patch on our code base which change the line which defines the behavior. https://gist.github.com/lotuc/fba421f74ea8610494118bc899f9f1b5#file-camel_snake_kebab_utils-clj-L35 |
becaouse
In my opinion numbers case should be recognised as a case of the preceding word
The text was updated successfully, but these errors were encountered: