Reduce length of hook names to <50

resolves #59

.pre-commit-config.yaml

@@ -22,3 +22,10 @@ repos:
  rev: 5.12.0
  hooks:
  - id: isort
+- repo: local
+ hooks:
+ - id: check-hook-name-length
+ name: 'Check hook name length'
+ language: python
+ entry: ./scripts/check-hook-name-length.py
+ files: ^\.pre-commit-hooks\.yaml$

.pre-commit-hooks.yaml

@@ -11,7 +11,7 @@
  language: python
  types: [text]
 - id: fix-spaces
- name: 'Fix irregular space characters by normalizing to "space"'
+ name: 'Normalize irregular space characters to "space"'
  description: 'Replace non-breaking spaces and other characters with the standard space character'
  entry: fix-spaces
  language: python

scripts/check-hook-name-length.py

@@ -0,0 +1,34 @@
+#!/usr/bin/env python
+import os
+import sys
+
+ROOTDIR = os.path.dirname(os.path.dirname(os.path.abspath(__file__)))
+HOOK_FILE = os.path.join(ROOTDIR, ".pre-commit-hooks.yaml")
+
+
+def iter_hook_names():
+ with open(HOOK_FILE) as fp:
+ for line in fp:
+ line = line.strip()
+ if line.startswith(("name: ", "- name: ")):
+ name = line.partition(":")[2].strip()
+ if name.startswith("'"):
+ yield name.strip("'")
+ elif name.startswith('"'):
+ yield name.strip('"')
+ else:
+ yield name
+
+
+def main():
+ success = True
+ for hook_name in iter_hook_names():
+ if len(hook_name) > 50:
+ print(f"ERROR: hook name too long: '{hook_name}'")
+ success = False
+ if not success:
+ sys.exit(1)
+
+
+if __name__ == "__main__":
+ main()