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

Added UPPER CASE and Human case. #48

Closed
wants to merge 4 commits into from

Conversation

azaleacolburn
Copy link

@azaleacolburn azaleacolburn commented Aug 14, 2023

Resolves issue #40

@jplatte
Copy link
Collaborator

jplatte commented Aug 15, 2023

Closing as per #40 (comment). I'm sorry about the wasted work, but I really don't think adding ever more casing conventions is a useful direction for this crate to go.

@jplatte jplatte closed this Aug 15, 2023
@azaleacolburn
Copy link
Author

Closing as per #40 (comment). I'm sorry about the wasted work, but I really don't think adding ever more casing conventions is a useful direction for this crate to go.

Don't worry, it wasn't much work, but you should probably change the README.
Screenshot 2023-08-15 at 2 16 10 AM

What direction do you think the crate should be taken? I'm more than happy to help with anything that can be done.

@jplatte
Copy link
Collaborator

jplatte commented Aug 15, 2023

Well, I think well-established is the important bit there. But I don't think that sentence is very useful anyways, probably makes sense to remove.

What direction do you think the crate should be taken? I'm more than happy to help with anything that can be done.

This might seem like a weird question but why do you want to contribute to this crate in particular?

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

Successfully merging this pull request may close these issues.

2 participants