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

Merge pull request #1684 from pre-commit/migrate_config_less_work

Browse source 
only perform migrate_config parsing if it is a list
This commit is contained in:
Anthony Sottile 2020-11-05 16:27:23 -08:00 • committed by GitHub
commit c972205214
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 10 additions and 10 deletions
Download patch file Download diff file Expand all files Collapse all files

20
pre_commit/commands/migrate_config.py
View file

@ -12,17 +12,17 @@ def _is_header_line(line: str) -> bool:
def _migrate_map(contents: str) -> str: 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
header = ''.join(lines[:i])
rest = ''.join(lines[i:])
if isinstance(yaml_load(contents), list): 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:])
# If they are using the "default" flow style of yaml, this operation # If they are using the "default" flow style of yaml, this operation
# will yield a valid configuration # will yield a valid configuration
try: try: