pre-commit/pre-commit-hooks
1
0
Fork 0
mirror of https://github.com/pre-commit/pre-commit-hooks.git synced 2026-04-05 19:46:54 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

[mixed_line_ending] convert a loop to max()

Browse source
This commit is contained in:
NotWearingPants 2021-10-12 20:57:44 +03:00 • committed by GitHub
parent f9ba5db3f1
commit 58487eb58b
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 2 additions and 8 deletions
Download patch file Download diff file Expand all files Collapse all files

10
pre_commit_hooks/mixed_line_ending.py
View file

@ -40,15 +40,9 @@ def fix_filename(filename: str, fix: str) -> int:
return mixed
if fix == 'auto':
max_ending = LF
max_lines = 0
# ordering is important here such that lf > crlf > cr
for ending_type in ALL_ENDINGS:
# also important, using >= to find a max that prefers the last
if counts[ending_type] >= max_lines:
max_ending = ending_type
max_lines = counts[ending_type]
# max prefers the first item when multiple items are the max
max_ending = max(reversed(ALL_ENDINGS), key=counts.get)
_fix(filename, contents, max_ending)
return 1
else: