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

Documentation: PascalCase is referred to as CamelCase #2849

Closed
gomesalexandre opened this issue Jan 13, 2020 · 4 comments
Closed

Documentation: PascalCase is referred to as CamelCase #2849

gomesalexandre opened this issue Jan 13, 2020 · 4 comments

Comments

@gomesalexandre
Copy link

Hi!
Currently in the rust docs naming conventions, it is recommended to use CamelCase for some constructs.
However, camel casing is usually camelCase, while such capitalizing the first letter capitalized too is typically referred to as PascalCase.
While it is not false per se(PascalCase can be seen as a subset of camelCase), it is confusing, and would be better and more accurate IMO to change occurrences of it to PascalCase.

Thanks 🤘

@shepmaster
Copy link
Member

This has already been discussed in #2389. The consensus was to use UpperCamelCase instead.

@shepmaster
Copy link
Member

shepmaster commented Jan 13, 2020

Currently in the rust docs naming conventions

Your link is the permalink to the 1.0.0 style guide: https://doc.rust-lang.org/1.0.0/style/style/naming/README.html. Please be cautious when using outdated reference material.

In fact, this has been removed from the official docs. I think the closest equivalent is https://rust-lang.github.io/api-guidelines/ and https://github.com/rust-dev-tools/fmt-rfcs

@gomesalexandre
Copy link
Author

@shepmaster That's amazing! 1.0.0 style guide is the Google search result currently when searching for naming conventions so I've figured out it was current.

@ghost
Copy link

ghost commented Feb 14, 2023

Terrible consensus! PascalCase was a great standard name, very established. Why not call it CamelCaseWithInitialCapital too then? UPPERCAMELCASE

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

No branches or pull requests

2 participants