third-party/pre-commit
1
0
Fork 0
mirror of https://github.com/pre-commit/pre-commit.git synced 2026-02-17 08:14:42 +04:00
Code Issues Projects Releases Packages Wiki Activity Actions

Remove Manifest, no longer a useful abstraction

Browse source
This commit is contained in:
Anthony Sottile 2017-10-26 16:08:40 -07:00
parent 71822279ee
commit 84b1ba520d
6 changed files with 38 additions and 90 deletions
Download patch file Download diff file Expand all files Collapse all files

11
pre_commit/repository.py
View file

@ -14,10 +14,10 @@ import pre_commit.constants as C
from pre_commit import five
from pre_commit import git
from pre_commit.clientlib import is_local_repo
from pre_commit.clientlib import load_manifest
from pre_commit.clientlib import MANIFEST_HOOK_DICT
from pre_commit.languages.all import languages
from pre_commit.languages.helpers import environment_dir
from pre_commit.manifest import Manifest
from pre_commit.prefixed_command_runner import PrefixedCommandRunner
from pre_commit.schema import apply_defaults
from pre_commit.schema import validate
@ -152,13 +152,14 @@ class Repository(object):
return self._cmd_runner
@cached_property
def manifest(self):
return Manifest(self._repo_path)
def manifest_hooks(self):
manifest_path = os.path.join(self._repo_path, C.MANIFEST_FILE)
return {hook['id']: hook for hook in load_manifest(manifest_path)}
@cached_property
def hooks(self):
for hook in self.repo_config['hooks']:
if hook['id'] not in self.manifest.hooks:
if hook['id'] not in self.manifest_hooks:
logger.error(
'`{}` is not present in repository {}. '
'Typo? Perhaps it is introduced in a newer version? '
@ -169,7 +170,7 @@ class Repository(object):
exit(1)
return tuple(
(hook['id'], _hook(self.manifest.hooks[hook['id']], hook))
(hook['id'], _hook(self.manifest_hooks[hook['id']], hook))
for hook in self.repo_config['hooks']
)