-
Notifications
You must be signed in to change notification settings - Fork 478
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
As someone reviewing code I don't want to be distracted by alternative styles for placing curly braces #4992
Milestone
Comments
This looks like what |
@pameyer |
@pdurbin Probably, but I'll find out for sure. |
Closed
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
All of the developers at IQSS use Netbeans to edit Dataverse code and at http://guides.dataverse.org/en/4.9.2/developers/coding-style.html#format-code-you-changed-with-netbeans we kindly ask contributors to format the code they changed (not all the code in the file they touched) with Netbeans. That way, people doing code review are not distracted by differences of style and can focus on how the behavior of the code is changing.
In the spirit of small chunks, I'd like to update the dev guide to specifically address placement of curly braces. I don't know how to express this in Checkstyle (or even what it's called), but here are the settings Netbeans uses for braces:
The text was updated successfully, but these errors were encountered: