Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Reconsider the naming approach #376

Open
kanarus opened this issue Feb 15, 2025 · 3 comments
Open

Reconsider the naming approach #376

kanarus opened this issue Feb 15, 2025 · 3 comments
Labels
rename Rename existing items

Comments

@kanarus
Copy link
Member

kanarus commented Feb 15, 2025

https://www.reddit.com/r/rust/s/vt1gQFqaQa

One petty nitpick: a thing that bothered me about this library is that it violates Rust's standardized naming convention:

In UpperCamelCase, acronyms and contractions of compound words count as one word: use Uuid rather than UUID, Usize rather than USize or Stdin rather than StdIn.

It always bothers me when certain libraries does this, because it's a bit hard to justify a breaking change for something like this, so it will continually stick out like a sore thumb...possibly forever.

@kanarus kanarus changed the title Reconsider the naming approach https://www.reddit.com/r/rust/s/vt1gQFqaQa Reconsider the naming approach Feb 15, 2025
@kanarus
Copy link
Member Author

kanarus commented Feb 17, 2025

builtin fangs

  • JWT -> Jwt
  • CORS -> Cors

@kanarus
Copy link
Member Author

kanarus commented Feb 20, 2025

builtin formats

  • JSON -> Json
  • HTML -> Html
  • URLEncoded -> UrlEncoded

@kanarus kanarus added the rename Rename existing items label Feb 22, 2025
@kanarus
Copy link
Member Author

kanarus commented Feb 25, 2025

openapi

  • security::APIKey -> security::ApiKey
  • security::OAuthFlow -> OauthFlow

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
rename Rename existing items
Projects
None yet
Development

No branches or pull requests

1 participant