-
Notifications
You must be signed in to change notification settings - Fork 28
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
Question: naming conventions #133
Comments
Heh, I am realizing that I associate |
Personally, I prefer |
I no longer remember why we chose a mix of camel caps and snake case in rust, just that it took some precedent from python. I think the outside world has often thought rust's convention is weird. It doesn't matter a lot to me, but I do find myself getting conventions wrong in other languages because I habitually use rust conventions. I think dada will initially attract curiosity from rust developers. It's a great opportunity to make different choices than rust where it matters. I think it probably doesn't matter here and will cause frustration and confusion (like it does when I switch between js and rust). |
I do want to emphasize though that I do not have a strong opinion. |
I think you're right that we have to choose carefully where we choose to stray from convention, both in comparison to Rust and in comparison to other languages. I don't exactly recall how we chose I think I'm persuaded that we should stick with Rust capitalization conventions, even if I've come to vaguely prefer I am now toying with the idea of a |
I am debating whether to use Rust-like
snake_case
naming or Java/JS likecamelCase
.On the merits, I think I would adopt camelCase:
foo.someField.someOtherField.share
is easier to visually separate thanfoo.some_field.some_other_field.share
(the.
and_
kind of run together for me)_
vs.
to be "roughly equivalent" in terms of people's ability to parse, but camel-case had a slight edge. I imagine it may be because of this phenomena.But, I want Dada and Rust to be good buddies, and I am reluctant to introduce an impedance mismatch.
Thoughts?
The text was updated successfully, but these errors were encountered: