Replies: 1 comment 2 replies
-
|
Hello @bocops That works for me. I'd also favor 4-space indentation over tabs and braces set as shown in your example. I'll check detekt configuration and enable checkstyle for Java code so that it is executed on pull request creation and merges into master. As well we could then trigger it by running a gradle check command locally. Does that sound ok? I've added #32 to the list of issues. |
Beta Was this translation helpful? Give feedback.
-
|
Sounds good! |
Beta Was this translation helpful? Give feedback.
-
|
Also added this: #34 Works complementarily to checkstyle. |
Beta Was this translation helpful? Give feedback.
-
I've noticed that code style isn't completely consistent in what we have so far, especially regarding indentation (tab vs. spaces, and how many of them), but probably also brace placement and related things.
I prefer OTBS with 4-space indentation, but wouldn't mind working with anything else if there are other preferences. We probably just should decide one way or the other early on, so that we don't re-indent each others code too often. ;) Any thoughts?
Beta Was this translation helpful? Give feedback.
All reactions