commit | 1e21f82b89be06ec790ed78534ef273587842685 | [log] [tgz] |
---|---|---|
author | Sorin Basca <sorinbasca@google.com> | Mon Sep 04 17:34:18 2023 +0000 |
committer | Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> | Mon Sep 04 17:34:18 2023 +0000 |
tree | 8d97a09c1f3796982f45d1b52ff7b1e0c40f54da | |
parent | 426ce13b1c31108dffa9c0a0068349fa8eff6ca6 [diff] | |
parent | e165db6c2e3aba210008bc636aadb92d0c66ae6a [diff] |
Move to checkstyle 10.12.3 am: 3213ec11cc am: e165db6c2e Original change: https://android-review.googlesource.com/c/platform/prebuilts/checkstyle/+/2735762 Change-Id: Id1dde064e541a5ccc400f29568affc97973661a0 Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
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.