Merge pull request #1684 from pre-commit/migrate_config_less_work

asottile · web-flow · commit c9722052145c · 2020-11-05T16:27:23.000-08:00
only perform migrate_config parsing if it is a list
diff --git a/pre_commit/commands/migrate_config.py b/pre_commit/commands/migrate_config.py
@@ -12,17 +12,17 @@ def _is_header_line(line: str) -> bool:
 
 
 def _migrate_map(contents: str) -> str:
-    # Find the first non-header line
-    lines = contents.splitlines(True)
-    i = 0
-    # Only loop on non empty configuration file
-    while i < len(lines) and _is_header_line(lines[i]):
-        i += 1
+    if isinstance(yaml_load(contents), list):
+        # Find the first non-header line
+        lines = contents.splitlines(True)
+        i = 0
+        # Only loop on non empty configuration file
+        while i < len(lines) and _is_header_line(lines[i]):
+            i += 1
 
-    header = ''.join(lines[:i])
-    rest = ''.join(lines[i:])
+        header = ''.join(lines[:i])
+        rest = ''.join(lines[i:])
 
-    if isinstance(yaml_load(contents), list):
         # If they are using the "default" flow style of yaml, this operation
         # will yield a valid configuration
         try:
