- Use appropriate formatting
- Java
- 4 Spaces (No tabs PLEASE)
- Brackets on the same line
- Space between all keywords, operators and values
- Long lines are OK - do not break long lines unless there is a logical, tabular break pattern (for example initialising elements in a long array) or if Vanilla does so
- Generally you should explicitly use
this. - Generally you should use braces { } following
if ()andfor ()statements, even if it is only one line. - Interfaces - do not put the prefix 'I' on the file name (example
Rotatable, notIRotatable)
- JSON
- 2 Spaces indent (NO TABS)
- Keep arrays on the same line (do not wrap)
- Spaces after
,and:
- Commits
- Use the Conventional Commits specification when creating commits.
- Pull Requests
- Pull request titles must use the Conventional Commits specification, just like commit names.
- Java
- Name all variables correctly
- All PRs require at least one successful review from a team member to merge
- Please do not submit simple PRs like minor readme changes or spelling fixes, unless they are a part of larger changes such as a new feature or bugfix.