Simplify tool execution and improve code documentation.
Review Request #8766 — Created Feb. 19, 2017 and discarded
Because we now use status updates for reporting Review Bot tool results, we
don't need to keep track of which files were processed or ignored. This change
simplifies that, and updates all the tools to include docstrings where
appropriate.
Change Summary:
Syntax
Commit: |
|
||||
|---|---|---|---|---|---|
Diff: |
Revision 2 (+153 -46) |
-
Tool: Pyflakes Processed Files: bot/reviewbot/tools/__init__.py bot/reviewbot/tools/pyflakes.py bot/reviewbot/tools/pmd.py bot/reviewbot/tools/cppcheck.py bot/reviewbot/tools/cpplint.py bot/reviewbot/tools/buildbot.py bot/reviewbot/tools/pep8.py bot/reviewbot/tools/jshint.py Tool: PEP8 Style Checker Processed Files: bot/reviewbot/tools/__init__.py bot/reviewbot/tools/pyflakes.py bot/reviewbot/tools/pmd.py bot/reviewbot/tools/cppcheck.py bot/reviewbot/tools/cpplint.py bot/reviewbot/tools/buildbot.py bot/reviewbot/tools/pep8.py bot/reviewbot/tools/jshint.py
Tool: PEP8 Style Checker Processed Files: bot/reviewbot/tools/__init__.py bot/reviewbot/tools/pyflakes.py bot/reviewbot/tools/pmd.py bot/reviewbot/tools/cppcheck.py bot/reviewbot/tools/cpplint.py bot/reviewbot/tools/buildbot.py bot/reviewbot/tools/pep8.py bot/reviewbot/tools/jshint.py