Do not hide user-specified skipped hooks

e.g.

$ SKIP=mypy pre-commit run --files pre_commit/*.py
Trim Trailing Whitespace.................................................Passed
Fix End of Files.........................................................Passed
Check docstring is first.................................................Passed
Check JSON...........................................(no files to check)Skipped
Check Yaml...........................................(no files to check)Skipped
Debug Statements (Python)................................................Passed
Tests should end in _test.py.........................(no files to check)Skipped
Fix requirements.txt.................................(no files to check)Skipped
Fix double quoted strings................................................Passed
flake8...................................................................Passed
autopep8.................................................................Passed
Validate Pre-Commit Manifest.........................(no files to check)Skipped
pyupgrade................................................................Passed
Reorder python imports...................................................Passed
Add trailing commas......................................................Passed
setup-cfg-fmt........................................(no files to check)Skipped
mypy....................................................................Skipped
Check hooks apply to the repository..................(no files to check)Skipped
Check for useless excludes...........................(no files to check)Skipped

Versus:

$ SKIP=mypy pre-commit run --files pre_commit/*.py -q
Trim Trailing Whitespace.................................................Passed
Fix End of Files.........................................................Passed
Check docstring is first.................................................Passed
Debug Statements (Python)................................................Passed
Fix double quoted strings................................................Passed
flake8...................................................................Passed
autopep8.................................................................Passed
pyupgrade................................................................Passed
Reorder python imports...................................................Passed
Add trailing commas......................................................Passed
mypy....................................................................Skipped

Here we still show mypy which the user had said to skip.
This commit is contained in:
Peter Cock 2020-08-24 19:19:04 +01:00
parent 7556297337
commit 662fdf2687

View file

@ -142,7 +142,7 @@ def _run_single_hook(
filenames = classifier.filenames_for_hook(hook)
if hook.id in skips or hook.alias in skips:
if not quiet:
# The user specified to skip this, ignore quiet mode
output.write(
_full_msg(
start=hook.name,