Skip to content
This repository has been archived by the owner on Oct 6, 2024. It is now read-only.

add lowerCamel #84

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

add lowerCamel #84

wants to merge 1 commit into from

Conversation

usrtax
Copy link

@usrtax usrtax commented Aug 13, 2022

No description provided.

@Amejonah1200
Copy link

Why is it still open?

@zabackary
Copy link

I know this is a little old, but is there a chance that this will ever be merged? Like someone mentioned in #72, I'm trying to use paste to generate JSON keys, so having a lower camel case converter would be useful to avoid having to do anything at runtime.

By the way, for the sake of clean code, the implementations for both :lowerCamel and :camel could be extracted to one function, since the only thing that needs to be changed is prev, which needs to be _ with camel and something else with lowerCamel. Also, this is a Rust library, so using snake_case (:lower_camel) might be better as the name of the modifier.

@adsick
Copy link

adsick commented Aug 6, 2024

I'd love to see this merged. Regarding the syntax: I feel like current "camel" should be renamed to pascal and the new "lower camel" should be named simply camel.

@its-the-shrimp
Copy link

I also needed this just now to generate types for interop with a remote API that uses JSON, would love to see this merged ❤️

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

Successfully merging this pull request may close these issues.

5 participants