RoboLoCo, Team 5338
Replies: 1 comment
-
|
Checks I suggest we add:
These wouldn't be configured to cause a compile error like spotless, however they would be flagged in a pull req as one of the checks. |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
One annoying thing about our code (that I am partially a contributor to) is that even though our general style is somewhat unified through spotless, the specific code itself does have discrepancies that I would like to rectify. For instance, our constants don't follow one naming format or another. Another issue is our branch naming, for which we don't have a consistent pattern.
We need to create stricter guidelines on formatting. I suggest that we use checkstyle to enforce this, and will look into installing it onto our repo soon.
Beta Was this translation helpful? Give feedback.
All reactions