commit | 78d505d154b0b3f05fb8306e7fc23121df1b6ea0 | [log] [tgz] |
---|---|---|
author | Xin Li <delphij@google.com> | Tue Sep 01 21:46:23 2020 +0000 |
committer | Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> | Tue Sep 01 21:46:23 2020 +0000 |
tree | d23c11abff6ec562d0b6bf17274f239af51c3b49 | |
parent | 202f9ee027eabdb0e186388ff843dfc28f328676 [diff] | |
parent | 44ed2c04586227c7a71ba9cdfa7a1f1538c72a01 [diff] |
[automerger skipped] Merge "Merge Android R (rvc-dev-plus-aosp-without-vendor@6692709)" into stage-aosp-master am: 2738924fcd -s ours am: 44ed2c0458 -s ours am skip reason: Change-Id I8749655bfbaf4d30f5ddfcba35d0eff2a66ec991 with SHA-1 a0e857181e is in history Original change: https://googleplex-android-review.googlesource.com/c/platform/prebuilts/checkstyle/+/12470199 Change-Id: Ide21a33a60e0087acd86d69ca5397ba3e559c7e8
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.