Rename check-files-matches-any to check-hooks-apply

asottile · asottile · commit e99813f117db · 2017-11-03T09:18:58.000-07:00
diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
@@ -27,5 +27,5 @@ repos:
     -   id: add-trailing-comma
 -   repo: meta
     hooks:
+    -   id: check-hooks-apply
     -   id: check-useless-excludes
-    -   id: check-files-matches-any
diff --git a/pre_commit/meta_hooks/check_hooks_apply.py b/pre_commit/meta_hooks/check_hooks_apply.py
diff --git a/pre_commit/repository.py b/pre_commit/repository.py
@@ -255,7 +255,7 @@ class MetaRepository(LocalRepository):
     @cached_property
     def manifest_hooks(self):
         # The hooks are imported here to prevent circular imports.
-        from pre_commit.meta_hooks import check_files_matches_any
+        from pre_commit.meta_hooks import check_hooks_apply
         from pre_commit.meta_hooks import check_useless_excludes
 
         def _make_entry(mod):
@@ -267,11 +267,11 @@ def _make_entry(mod):
 
         meta_hooks = [
             {
-                'id': 'check-files-matches-any',
-                'name': 'Check hooks match any files',
+                'id': 'check-hooks-apply',
+                'name': 'Check hooks apply to the repository',
                 'files': '.pre-commit-config.yaml',
                 'language': 'system',
-                'entry': _make_entry(check_files_matches_any),
+                'entry': _make_entry(check_hooks_apply),
             },
             {
                 'id': 'check-useless-excludes',
diff --git a/tests/meta_hooks/check_hooks_apply_test.py b/tests/meta_hooks/check_hooks_apply_test.py
@@ -1,6 +1,6 @@
 from collections import OrderedDict
 
-from pre_commit.meta_hooks import check_files_matches_any
+from pre_commit.meta_hooks import check_hooks_apply
 from pre_commit.util import cwd
 from testing.fixtures import add_config_to_repo
 from testing.fixtures import git_dir
@@ -25,7 +25,7 @@ def test_hook_excludes_everything(
     add_config_to_repo(repo, config)
 
     with cwd(repo):
-        assert check_files_matches_any.main(()) == 1
+        assert check_hooks_apply.main(()) == 1
 
     out, _ = capsys.readouterr()
     assert 'check-useless-excludes does not apply to this repository' in out
@@ -50,7 +50,7 @@ def test_hook_includes_nothing(
     add_config_to_repo(repo, config)
 
     with cwd(repo):
-        assert check_files_matches_any.main(()) == 1
+        assert check_hooks_apply.main(()) == 1
 
     out, _ = capsys.readouterr()
     assert 'check-useless-excludes does not apply to this repository' in out
@@ -75,7 +75,7 @@ def test_hook_types_not_matched(
     add_config_to_repo(repo, config)
 
     with cwd(repo):
-        assert check_files_matches_any.main(()) == 1
+        assert check_hooks_apply.main(()) == 1
 
     out, _ = capsys.readouterr()
     assert 'check-useless-excludes does not apply to this repository' in out
@@ -100,7 +100,7 @@ def test_hook_types_excludes_everything(
     add_config_to_repo(repo, config)
 
     with cwd(repo):
-        assert check_files_matches_any.main(()) == 1
+        assert check_hooks_apply.main(()) == 1
 
     out, _ = capsys.readouterr()
     assert 'check-useless-excludes does not apply to this repository' in out
@@ -124,7 +124,7 @@ def test_valid_includes(
     add_config_to_repo(repo, config)
 
     with cwd(repo):
-        assert check_files_matches_any.main(()) == 0
+        assert check_hooks_apply.main(()) == 0
 
     out, _ = capsys.readouterr()
     assert out == ''
