Add [project/foo/bar] to the beginning of each Checkstyle error line
if the REPO_PROJECT environmental variable is set (set by repo hooks).
Test: manually attempted to run checkstyle.
Change-Id: I8ccf76d9ce9306668109029267081924300cb6ce
Certain projects might want to enable checkstyle gradually so
we need a way to whitelist which files should be checked.
Test: added test_FilterFiles
Change-Id: I501677ce824f882ccb5f2ad129af8f596d01c6e8
Bug: None
Test: Existing test_ShouldSkip will test this as it is using
checkstyle.SKIPPED_RULES_FOR_TEST_FILES
Change-Id: I56c8c86a36f93b6b72a87fd97b75e5233fde9077
This also requires modifying the logic to create temporary files so that
the filename is the same as that in source tree.
Bug: 27109166
Change-Id: I73f2406d5c85d45b0b259f0fd7dd081e44df029c
Usage e.g.: ./checkstyle.py -s 3aca02ca91816a86febb5ed6e380ec2122adea47
Additionally this CL adds a pre-push script that can be used a git
hook to run Checkstyle on 'repo upload'
Bug: 25852971
Change-Id: Ia00d48df80b2b024de0899d9590868016c5b7bf0
It is valid to have empty modified_lines list for files
that only contain deletions. In such case we should skip
all the errors except for forced rules.
Updated the test to match the expectation.
Change-Id: I6993968b882fb6fbe2ba1f63f3b6879c3308ff34
Checkstyle.py was not handling additions of new files resulting in
a crash. For new files git.modified_lines() returns None. Added code
handles that case.
Bug: 25852971
Change-Id: I91e2b8c92581ec9e89bcbbcd2d274f56c791f3a9
(cherry picked from commit 5b87dbad0c)
Allow developers to run Java style check in two ways.
Specific file:
checkstyle.py -f path/to/file/View.java
Lines modified in the latest commit:
checkstyle.py
Review moved from https://googleplex-android-review.googlesource.com/#/c/818813/
due to move from packages/experimental to development/tools
Bug: 25852971
Change-Id: Ibcf9d80644f2938828b21e8f50ab0d6f39c286b0
(cherry picked from commit 96f27d0960)
