Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit e261526d authored by Jordan Demeulenaere's avatar Jordan Demeulenaere
Browse files

Add a ktfmt hook for some SystemUI files 

This CL enables a ktfmt hook to check that Kotlin files are properly
formatted.

The set of files or directories that should be included is specified in
a simple ktfmt_includes.txt file. That way, existing files that are not
properly formatted yet won't be checked and uploaders won't be bothered
by this check. In the future, we will check all files created after some
baseline.

At the moment, only Kotlin files that I know are already formatted with
ktfmt were included in that list.

Test: Manual
Bug: 235461679
Change-Id: Idaed15c39f6af413c305f4c74f142a2895b5a753
parent eddeda4f

PREUPLOAD.cfg

+2 −0
Original line number Diff line number Diff line
@@ -26,6 +26,8 @@ hidden_api_txt_checksorted_hook = ${REPO_ROOT}/tools/platform-compat/hiddenapi/c 


hidden_api_txt_exclude_hook = ${REPO_ROOT}/frameworks/base/tools/hiddenapi/exclude.sh ${PREUPLOAD_COMMIT} ${REPO_ROOT}



ktfmt_hook = ${REPO_ROOT}/external/ktfmt/ktfmt.py --check -i ${REPO_ROOT}/frameworks/base/ktfmt_includes.txt ${PREUPLOAD_FILES}



ktlint_hook = ${REPO_ROOT}/prebuilts/ktlint/ktlint.py -f ${PREUPLOAD_FILES}



owners_hook = ${REPO_ROOT}/frameworks/base/tools/aosp/aosp_sha.sh ${PREUPLOAD_COMMIT} "OWNERS$"

ktfmt_includes.txt

0 → 100644
+9 −0
Original line number Diff line number Diff line 
packages/SystemUI/compose/

packages/SystemUI/screenshot/

packages/SystemUI/src/com/android/systemui/people/data

packages/SystemUI/src/com/android/systemui/people/ui

packages/SystemUI/src/com/android/systemui/keyguard/data

packages/SystemUI/src/com/android/systemui/keyguard/dagger

packages/SystemUI/src/com/android/systemui/keyguard/domain

packages/SystemUI/src/com/android/systemui/keyguard/shared

packages/SystemUI/src/com/android/systemui/keyguard/ui
 
 No newline at end of file