diff options
| author | Luis Hector Chavez <lhchavez@google.com> | 2016-12-22 14:07:03 -0800 |
|---|---|---|
| committer | Luis Hector Chavez <lhchavez@google.com> | 2016-12-22 14:07:03 -0800 |
| commit | d1fd88b71ce4b8b1b65de1081a592510e259fc2c (patch) | |
| tree | a9662a2e6651be7b0f9608001b0b1b785adbd9ef /rh | |
| parent | dab680c7231cc2f0169d847ed087dc602e58e781 (diff) | |
| download | repohooks-d1fd88b71ce4b8b1b65de1081a592510e259fc2c.tar.gz | |
Add support for automatic fixups to google-java-format
Similar to git-clang-format, google-java-format supports the --fix flag
and can trivially auto-fixup style violations.
Bug: None
Test: Ran hook
Change-Id: Ia9a9aaab1a56a7db68030c6ca20289ec88ac8aa6
Diffstat (limited to 'rh')
| -rw-r--r-- | rh/hooks.py | 11 |
1 files changed, 7 insertions, 4 deletions
diff --git a/rh/hooks.py b/rh/hooks.py index 47926bc..4e456e1 100644 --- a/rh/hooks.py +++ b/rh/hooks.py @@ -315,10 +315,13 @@ def check_google_java_format(project, commit, _desc, _diff, options=None): tool = get_helper_path('google-java-format.py') google_java_format = options.tool_path('google-java-format') google_java_format_diff = options.tool_path('google-java-format-diff') - cmd = [tool, '--google-java-format', google_java_format, - '--google-java-format-diff', google_java_format_diff, - '--commit', commit] + options.args() - return _check_cmd('google-java-format', project, commit, cmd) + tool_args = ['--google-java-format', google_java_format, + '--google-java-format-diff', google_java_format_diff, + '--commit', commit] + options.args() + cmd = [tool] + tool_args + fixup_func = _fixup_func_caller([tool, '--fix'] + tool_args) + return _check_cmd('google-java-format', project, commit, cmd, + fixup_func=fixup_func) def check_commit_msg_bug_field(project, commit, desc, _diff, options=None): |