-
Notifications
You must be signed in to change notification settings - Fork 52
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
Curly braces #43
Comments
@jakerella |
Yes, this would be great to see it for languages other than Java. I'm very curious to see the result for C#. I have a feeling that this is one of the few languages where "next line" is more popular. |
@outsideris Aah! Yes, I did not see the Java one before. I would like to see that for JavaScript. |
@mlaflamm It is already added in C#. Your feeling is right! 😄 @jakerella I see what you mean. |
This is so much needed for JS. |
I would love also to see the usage of curly braces against not to use curly braces at all, for 1 line statements, but since this only analyses line by line don't think this is possible |
I'd love to see the split of code with curly braces on the same line as functions or conditionals versus on the next line. (I'm a JavaScript dev, but seems like it would work in a lot of languages.)
Same line:
Next line:
The text was updated successfully, but these errors were encountered: