Clone this repo:
  1. 9c3ea5f Add dagger to import order else it conflicts with the aosp ordering followed by google-java-format by Anubha Kushwaha · 4 weeks ago master
  2. fd988c1 Add jvmTest to know list of test directories am: 6b2c40c903 am: d70d12e041 am: 7618fd3dd0 by Aurimas Liutikas · 9 months ago android13-dev android13-mainline-go-adservices-release android13-mainline-go-odp-release aml_go_ads_330913000 aml_go_odp_330912000 t_frc_ase_330444010
  3. 7618fd3 Add jvmTest to know list of test directories am: 6b2c40c903 am: d70d12e041 by Aurimas Liutikas · 9 months ago
  4. d70d12e Add jvmTest to know list of test directories am: 6b2c40c903 by Aurimas Liutikas · 9 months ago
  5. 6b2c40c Add jvmTest to know list of test directories by Aurimas Liutikas · 9 months ago androidx-customview-customview-poolingcontainer-release

Checkstyle

Checkstyle is used by developers to validate Java code style and formatting, and can be run as part of the pre-upload hooks.

Running it

It can be invoked in two ways.

  1. To check style of entire specific files: checkstyle.py -f FILE [FILE ...]
  2. To check style of the lines modified in the latest commit: checkstyle.py

Projects used

Checkstyle

A development tool to help programmers write Java code that adheres to a coding standard.

Git-Lint

Git-lint is a tool to run lint checks on only files changed in the latest commit.

  • URL: https://github.com/sk-/git-lint/
  • Version: 0.0.8
  • License: Apache 2.0
  • License File: gitlint/LICENSE
  • Local Modifications:
    • Downloaded gitlint/git.py and git/utils.py files individually.

Pre-upload linting

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.

Disabling Clang Format 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.