Merge pull request #387 from pre-commit/better_suggest_autoupdate

Clarify language around missing hooks.  Resolves #386
This commit is contained in:
Anthony Sottile
2016-06-25 10:52:32 -07:00
committed by GitHub
2 changed files with 6 additions and 2 deletions

View File

@@ -78,7 +78,9 @@ class Repository(object):
logger.error(
'`{0}` is not present in repository {1}. '
'Typo? Perhaps it is introduced in a newer version? '
'Often `pre-commit autoupdate` fixes this.'.format(
'Often you can fix this by removing the hook, running '
'`pre-commit autoupdate`, '
'and then adding the hook.'.format(
hook['id'], self.repo_config['repo'],
)
)

View File

@@ -556,7 +556,9 @@ def test_hook_id_not_present(tempdir_factory, store, fake_log_handler):
assert fake_log_handler.handle.call_args[0][0].msg == (
'`i-dont-exist` is not present in repository {0}. '
'Typo? Perhaps it is introduced in a newer version? '
'Often `pre-commit autoupdate` fixes this.'.format(path)
'Often you can fix this by removing the hook, '
'running `pre-commit autoupdate`, '
'and then adding the hook.'.format(path)
)