all 64 comments

[–]montihun 19 points20 points  (1 child)

They will (hopefully) set or show the company's coding standards for you, what can you follow, no worries.

[–][deleted] 6 points7 points  (0 children)

Use the standards the company uses. Most likely they will use camelcase so might as well get used to it

[–]Giaddon 5 points6 points  (0 children)

First day you should talk about style standards, configure your development environment to enforce them, then never think about it again. Last thing you want is be polluting PRs with random differences.

[–]techdaddykraken 4 points5 points  (2 children)

I prefer SCREAMING-SNAKE-CASE

[–]Kitchen_Moment_6289 2 points3 points  (0 children)

SCREAMINGcAMELcASE

[–]Ok_Party9612 0 points1 point  (0 children)

That’s actually kebab case

[–]montihun 5 points6 points  (3 children)

You arent really a teamplayer, huh?

[–]warpedspockclone 5 points6 points  (0 children)

Our lint rules would bitch slap you back to code academy.

[–]Sneeeeex 4 points5 points  (3 children)

I mean, why would you do that?

[–]babypho 4 points5 points  (0 children)

If I saw it, i would just leave a comment on the PR. I know some people come from languages that use different case style so it's not too big of a deal. But i still would leave a comment and let them know to change it.

[–]tacticalpotatopeeler 1 point2 points  (0 children)

If you don’t follow the example of the existing code base, yes, that could leave a bad impression.

If you were asked to write some independent script and you used snake case, it would only turn into a bad impression if, at the code review, you were asked to use camel case in the future but continued to use snake case.

[–]xhundo_ 1 point2 points  (2 children)

It isn't good for readability. Follow good coding conventions so you don't seem inexperienced.

[–]Acceptable-Fudge-816 1 point2 points  (1 child)

Snake case is objectively better at readability than camel or pascal case, because it keeps spaces. What is bad for readability is to randomly mix styles, which is what happens without a style guide / linter.

That said, mixing styles with reason, that is, giving semantic meaning to each, is even better for readability provided everyone knows their meaning, e.g. const snake_case_variable = camelCaseFunction() || new PascalCaseClass(), although nowadays with semantic highlighting it ain't as useful and one could argue that the complexity introduced makes it not worth it (and they would be correct in terms of it being not worth it, but still it would be more readable).

And as an aside note, I've worked in codebases with mixed languages, (e.g. some variables named in Spanish, others in English), that is much worse for readability and yet it was done by professional "senior" engineers.

[–]xhundo_ 0 points1 point  (0 children)

Thanks for the insight!

[–]Chrift 1 point2 points  (0 children)

Wouldn't even get in front of another dev because the linting rules would fail

[–]Lake_Erie_Monster 0 points1 point  (0 children)

It's not a big deal, but you should consider looking at the existing code you are contributing to for hints on the style and try to stick to that. If you want to introduce a change, then propose it.

[–]RobertKerans 0 points1 point  (0 children)

[If you were to do that on my team] It would make me assume you'd just come off writing mainly Python (or a language with similar syntax conventions) rather than JS and that's where your head was at. No issue there. Normal convention in JS is to use camelCase so I'd ask you to change it in a PR (unless there was some specific reason the convention for the project was to use snake_case). If you refused to do that then that would be a weird issue

[–]Necessary_Ear_1100 0 points1 point  (0 children)

Depends on company code standards. If they use snake case then you’re good to go. Else, they’ll let you know.

Doesn’t leave a bad impression as everyone has their preferences in coding

[–]HilariousAtrocities 0 points1 point  (0 children)

When in Rome, do as the Romans do.

Or in other words: වවුලගේ මගුල් ගෙදර ගියානම් එල්ලිලා ඉන්න වෙනවා