opendev/gerrit
Code Issues Proposed changes

Document Eclipse's warning free code policy

Browse Source 
Change-Id: I0429d00d693619cc56cb84d845ff1fe454441971
This commit is contained in:
David Ostrovsky 2017-07-06 06:53:24 +02:00
parent aa354e9365
commit 8eefe46f06
1 changed files with 11 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
Documentation/dev-contributing.txt
View File

@ -147,6 +147,17 @@ Password tab of the user settings page].
[[style]]
=== Style
This project has a policy of Eclipse's warning free code. Eclipse
configuration is added to git and we expect the changes to be
warnings free.
We do not ask you to use Eclipse for editing, obviously. We do ask you
to provide Eclipse's warning free patches only. If for some reasons, you
are not able to set up Eclipse and verify, that your patch hasn't
introduced any new Eclipse warnings, mention this in a comment to your
change, so that reviewers will do it for you. Yes, the way to go is to
extend gerrit CI to take care of this, but it's not yet implemented.
Gerrit generally follows the
link:https://google.github.io/styleguide/javaguide.html[Google Java Style
Guide].