commit | de2842a0f468084779d71164b1e5bf5079f65d48 | [log] [tgz] |
---|---|---|
author | Hayden Gomes <haydengomes@google.com> | Fri Aug 16 17:21:44 2019 -0700 |
committer | android-build-merger <android-build-merger@google.com> | Fri Aug 16 17:21:44 2019 -0700 |
tree | da7f6725407d4ece9fd372ce4ce8406d5dade979 | |
parent | 03087a617bfe55f862cb1ef43273a2bd08e8b6d6 [diff] | |
parent | 7d6bb91e31d18eadfaf855a9fb7ad6ba81b8b6d9 [diff] |
Adding Pre-upload instructions to checkstyle README am: 153a9a100e am: 7d6bb91e31 Change-Id: If7b157838d65b997a2f55243f4a6ffc0407f5488
Checkstyle is used by developers to validate Java code style and formatting, and can be run as part of the pre-upload hooks.
It can be invoked in two ways.
checkstyle.py -f FILE [FILE ...]
checkstyle.py
A development tool to help programmers write Java code that adheres to a coding standard.
Git-lint is a tool to run lint checks on only files changed in the latest commit.
To run checkstyle as part of the pre-upload hooks, add the following line to your PREUPLOAD.cfg
:
checkstyle_hook = ${REPO_ROOT}/prebuilts/checkstyle/checkstyle.py --sha ${PREUPLOAD_COMMIT}
Note that checkstyle does not always agree with clang-format, and so it's best to only have one enabled for Java.
In .clang-format
add the following to disable format checking and correcting for Java:
--- Language: Java DisableFormat: true SortIncludes: false ---
In some versions of clang-format, DisableFormat
doesn't stop the sorting of includes. So to fully disable clang-format from doing anything for Java files, both options are needed.