opendev/gerrit
Code Issues Proposed changes

Document one argument per line for constructor injection

Browse Source 
Change-Id: I4a698f6a3a1d601b691463ceaaeab407901c6a9d
This commit is contained in:
Saša Živkov 2015-02-10 09:58:47 +01:00
parent b7482d1829
commit c05a12f473
1 changed files with 1 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
Documentation/dev-contributing.txt
View File

@ -221,6 +221,7 @@ Here are some guidelines that Gerrit uses:
should be before the instance members. should be before the instance members.
* Annotations should go before language keywords (`final`, `private`, etc) + * Annotations should go before language keywords (`final`, `private`, etc) +
Example: `@Assisted @Nullable final type varName` Example: `@Assisted @Nullable final type varName`
* The `@Inject`-ed constructor arguments should be listed one per line.
* Imports should be mostly alphabetical (uppercase sorts before * Imports should be mostly alphabetical (uppercase sorts before
all lowercase, which means classes come before packages at the all lowercase, which means classes come before packages at the
same level). same level).