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

Go back to optional. Requires less changes to existing code.

Browse source
This commit is contained in:
Pedro Algarvio 2018-12-17 12:05:55 +00:00
parent b096c0b8f2
commit afbc57f2ad
No known key found for this signature in database
GPG key ID: BB36BF6584A298FF
1 changed files with 2 additions and 17 deletions
Download patch file Download diff file Expand all files Collapse all files

19
pre_commit/clientlib.py
View file

@ -29,20 +29,6 @@ def _make_argparser(filenames_help):
return parser return parser
class OptionalAlias(object):
def check(self, dct):
if 'alias' in dct:
cfgv.check_string(dct['alias'])
def apply_default(self, dct):
if 'alias' not in dct:
dct['alias'] = dct['id']
def remove_default(self, dct):
pass
MANIFEST_HOOK_DICT = cfgv.Map( MANIFEST_HOOK_DICT = cfgv.Map(
'Hook', 'id', 'Hook', 'id',
@ -50,7 +36,7 @@ MANIFEST_HOOK_DICT = cfgv.Map(
cfgv.Required('name', cfgv.check_string), cfgv.Required('name', cfgv.check_string),
cfgv.Required('entry', cfgv.check_string), cfgv.Required('entry', cfgv.check_string),
cfgv.Required('language', cfgv.check_one_of(all_languages)), cfgv.Required('language', cfgv.check_one_of(all_languages)),
cfgv.OptionalNoDefault('alias', cfgv.check_string), cfgv.Optional('alias', cfgv.check_string, ''),
cfgv.Optional( cfgv.Optional(
'files', cfgv.check_and(cfgv.check_string, cfgv.check_regex), '', 'files', cfgv.check_and(cfgv.check_string, cfgv.check_regex), '',
@ -140,7 +126,6 @@ CONFIG_HOOK_DICT = cfgv.Map(
'Hook', 'id', 'Hook', 'id',
cfgv.Required('id', cfgv.check_string), cfgv.Required('id', cfgv.check_string),
OptionalAlias(),
# All keys in manifest hook dict are valid in a config hook dict, but # All keys in manifest hook dict are valid in a config hook dict, but
# are optional. # are optional.
@ -149,7 +134,7 @@ CONFIG_HOOK_DICT = cfgv.Map(
*[ *[
cfgv.OptionalNoDefault(item.key, item.check_fn) cfgv.OptionalNoDefault(item.key, item.check_fn)
for item in MANIFEST_HOOK_DICT.items for item in MANIFEST_HOOK_DICT.items
if item.key not in ('id', 'alias') if item.key != 'id'
] ]
) )
CONFIG_REPO_DICT = cfgv.Map( CONFIG_REPO_DICT = cfgv.Map(