Closed tyronewilson closed 3 years ago
Just following up here. Are we able to merge this?
Thanks for this and sorry for the delay! Much appreciated and seems like a handy feature.
No worries, glad to help 👍
On Mon, 23 Nov 2020, 01:48 iancoleman, notifications@github.com wrote:
Thanks for this and sorry for the delay! Much appreciated and seems like a handy feature.
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/iancoleman/strcase/pull/28#issuecomment-731868062, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAOQI4QI4FF5QWUAKKRTTKTSRGPOFANCNFSM4SUITI7A .
I was using
strcase
to normalise some JSON keys and found that I had an acronym which wasn't being handled nicely. The acronym was "COGS" and it was being used to convert to either Cogs or cOGS. I wanted cogs.With this PR I can now do
this now means that using
ToLowerCamel("COGS")
will give me what I need.I think this is a useful feature in general as I found that the acronyms were currently limited to "ID": "id" with no way to configure additional ones.
Let me know if you would like to include this :)