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

preserveConsecutiveUppercase #29

Closed
donkeyDau opened this issue Nov 26, 2020 · 2 comments · Fixed by #30
Closed

preserveConsecutiveUppercase #29

donkeyDau opened this issue Nov 26, 2020 · 2 comments · Fixed by #30

Comments

@donkeyDau
Copy link
Contributor

In the latest release of https://github.com/sindresorhus/camelcase the option preserveConsecutiveUppercase was introduced. I need a comparable option for decamelize.

@sindresorhus Do you see that in the scope of this package?

I tried to fiddle around with the code but there would be probably some edge cases to be handled and more options to be introduced to meet the complexity of this. This would kind of ruin the the minimalist beauty of the code ...

@sindresorhus
Copy link
Owner

You mean you want fooBAR => foo_BAR, right? I guess that makes sense. But it would have to be a good PR that came with lots of tests, docs, and types.

@donkeyDau
Copy link
Contributor Author

donkeyDau commented Nov 27, 2020

You mean you want fooBAR => foo_BAR, right?

Yes.

But it would have to be a good PR that came with lots of tests, docs, and types.

I'll give it a try.

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

Successfully merging a pull request may close this issue.

2 participants