-
Notifications
You must be signed in to change notification settings - Fork 34
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
Added AsXxxCase wrappers #33
Conversation
I just realized I am using an old base. I'll rebase it later after the changes are approved. |
I like the idea, though don't have time to review right now. Will try to get to this soon-ish. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry for the long delay. Changes are looking good!
Rebased. |
pub use title::{AsTitleCase, ToTitleCase}; | ||
pub use upper_camel::{ | ||
AsUpperCamelCase, AsUpperCamelCase as AsPascalCase, ToPascalCase, ToUpperCamelCase, | ||
}; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would have actually preferred simply doing pub use upper_camel::*;
here though.
When are you going to publish a release containing this update? |
I'm not sure it makes sense to release 0.4.0 before #26 is done, but this project is overall very low on my todo list. I guess this could go into 0.3.4 though. Can you rebase this on top of the v0.3.x branch and open another PR with that? If you care, finishing #27 would be another nice to have for v0.3.x, seems like the original author has disappeared. |
Given
camel: &str, snake: &str
, the following line allocates 3String
s:Instead, I can write this to allocate only one
String