summaryrefslogtreecommitdiff
path: root/PREUPLOAD.cfg
AgeCommit message (Collapse)Author
2020-07-10Revert "Temporarily disable checkstyle hook"Aurimas Liutikas
This reverts commit 8497789b3b3f78e8eec53e946654afd896436c60. Reason for revert: This was fixed, see b/160796554 Bug: 160796554 Change-Id: I42f8d6b76c7319522239517ac3aff6eee7d9084b
2020-07-09Temporarily disable checkstyle hookAurimas Liutikas
checkstyle uses inefficient git calls that make it really slow for --partial-clone Bug: 160796554 Change-Id: I3bb2ac1bebdb6f778b2d2ff159eefb27664aa177
2020-04-06Appops permission monitoring for GET_USAGE_STATS.Alex Buynytskyy
This makes sure DataLoader won't be able to obtain read logs once user denies access. Bug: b/152633648 Test: atest PackageManagerShellCommandTest PackageManagerShellCommandIncrementalTest IncrementalServiceTest Test: adb shell appops set 1000 GET_USAGE_STATS deny Change-Id: Ibbb74933b4ef0dd8f5fe27732743e5820b8ee4dc
2020-01-27Enable clang-format for f/b/services/core/jniSiarhei Vishniakou
To avoid code review comments about code style, line length, and spacing, enable clang-format for frameworks/base/services/core/jni Bug: none Test: repo upload Change-Id: Ie38ecca2d5face78367d3eab450e497120ce12b1
2020-01-03Enable clang-format for core/jniSiarhei Vishniakou
To have consistent code style and avoid format-related comments in code review, enable clang-format in frameworks/base/core/jni. Bug: none Test: repo upload Change-Id: Ie9e5408d85036dfabae3a9c60b238b4a190da5fc
2019-12-10Enable clang-format for input codeSiarhei Vishniakou
Currently, there is no format enforcement for hid command. Add clang-format hook for all input-related code in frameworks/base. Bug: None Test: repo upload Change-Id: I08b8c6d1bae7d1954847c207ef8e6d80373b2d05
2019-11-01API Lint: remove obsolete python linterAdrian Roos
Linting is performed by metalava as part of the build now. Test: n/a Change-Id: I6223549657912fdf305a17f66f55b97561f7f7ac
2019-07-24Add ktlint to preupload hooksRemi NGUYEN VAN
The kotlin linter is particularly useful for tests written in Kotlin in frameworks/base. This is already merged in internal master. Test: Tried uploading change with/without kotlin style problems. Merged-In: Ie8659765b674ac7b2d82ed3d343f387195c07d83 Change-Id: I420366c49d071e022b8988c3d6293dbc8e9a283a
2018-12-10Enable checkstyle everywhere in frameworks/base.Lorenzo Colitti
This was done in internal master in ag/5397886. Do it in AOSP as well so we can benefit from checkstyle in AOSP as well. Test: None Change-Id: Ib00fc8d318e8127224846e0a69cb542178631437 Merged-In: I7d350c2bbb9fcd1019abd1addb035ce5e9e05acc
2018-11-29Prevent hiddenapi entries being added for libcore related projectsPaul Duffin
The libcore related projects (see below) have been (mostly) switched over to use UnsupportedAppUsage annotations, This change will prevent entries for those projects being added to a config/hiddenapi-* file. * libcore * external/bouncycastle * external/conscrypt * external/icu * external/okhttp * external/libphonenumber - still has a couple of entries in config/hiddenapi-light-greylist.txt due to limitations in UnsupportedAppUsage/class2greylist. Tested by attempting to upload the file with entries for libcore projects and without those entries and checking that the behavior is expected. Test: see above Bug: 117818301 Change-Id: I67a30b307e12e842b28cfb2160fab0029868fa06
2018-11-21Added OWNERS to Shell and hook to make sure it's developed on AOSP.Felipe Leme
Bug: none Test: none Change-Id: I1f6f554cf98c78d1ab8a1e820d485a37238230f5
2018-09-11Upload hook that guides people towards AOSP.Jeff Sharkey
The canonical location of certain parts of the source tree live in AOSP, and should not be changed internally to avoid merge conflicts. This initially starts with "OWNERS" files, but the hook can easily be repeated to match any files based on file path regexes. Bug: 113136846 Test: manual Change-Id: I113e63b6133f20389d84ec0d6d8b81cdbdf35d38 Merged-In: I113e63b6133f20389d84ec0d6d8b81cdbdf35d38
2018-04-24Merge "Add presubmit hook for hidden API lists." into pi-devMathew Inwood
am: 02033386c2 Change-Id: I6c437a6ffa62794f1fc88d0c8bcdcef798b68d8c
2018-04-24Add presubmit hook for hidden API lists.Mathew Inwood
Check that they're sorted as expected and contain no duplicates. The sort order now uses: $ LC_COLLATE=C sort -f So that non-alphanumeric characters are not ignored, giving a more intuitive sort order. the '-f' means ignore case. Also sort the existing lists accordingly. Test: repo upload Bug: 64382372 Merged-In: I52b884da33a9a46455df6747a215683d9d3c3218 Change-Id: I4cdd3bc5c11be91a9a3f678580af49ac67f3c968
2018-03-27Merge "Add linter for strings.xml." into pi-devJeff Sharkey
am: 642c0be3f3 Change-Id: I9677257b4ddfe5c1347b6de13b98e2b25fd649ac
2018-03-26Add linter for strings.xml.Jeff Sharkey
Strings are tedious to get right, so write a lint script that catches common errors to warn developers before they're uploaded. Test: builds, boots Bug: 76097999 Change-Id: I9826ca796c17cd93a100951d56214653de63a379
2018-03-23Merge "Enable Checkstyle for "repo upload" for more projects." am: dd6764a522Jack Yu
am: 2ff91e5132 Change-Id: Ie880f61f511ad51011607c531c9028376e5683d1
2018-03-22Enable Checkstyle for "repo upload" for more projects.Jack Yu
This CL enables Checkstyle for the following directories: - telephony/ Bug: 34746646, 32954735 Test: Compilation Change-Id: Ia67eaf5b871e6b74f21344fef1572835801843a5
2018-02-09Add checkstyle check to graphics packageSiyamed Sinir
Test: None Change-Id: I1c8342dda2678917e245e48ab141b2953a522197
2017-11-13Add api lint to upload checksJason Monk
Currently will never block upload, only considered a warning, but will print out any lint issues discovered. Test: do an upload Bug: 69115822 Change-Id: I69b8e5a3f92ac28ee3c954408e88a1056c33af65
2017-04-28Enable Checkstyle for "repo upload" for more projects.Aurimas Liutikas
This CL enables Checkstyle for the following directories: - core/java/android/ - core/tests/coretests/src/android/ Bug: 34746646 Test: None Change-Id: Ib8b3f198000be19e9a7a61aa5abd61d827454eba
2016-12-15Point to the new checkstyle location for frameworks/base PREUPLOAD hook.Aurimas Liutikas
This change depends on r.android.com/314437 Bug: 33666900 Test: None Change-Id: I301723d516803146943f0866ea35b68cc36af6af
2016-10-20Check USB and Print Framework code.Philip P. Moltmann
Test: Uploaded Change-Id: Ib103a4fdc1dea0efc3a1a1be8c864541d283343a
2016-10-12Enable Checkstyle preupload hook for certain frameworks/base directories.Aurimas Liutikas
Enables Checkstyle preupload hook for the following directories: - core/java/android/animation/ - core/java/android/text/ - core/java/android/view/ - core/java/android/transition/ - core/java/android/widget/ Test: tested manually uploading a CL that contained many changes to directories outside of the listed ones and checkstyle did not complain about those. Change-Id: I795ba478fad7af124c43ca1e18d51b33d7800976