mirror of
https://github.com/pre-commit/pre-commit-hooks.git
synced 2026-04-07 20:26:54 +00:00
Add parameter controlling whether lfs files are excluded from rule
Defaults to `true`, permitting large files as long as they are in lfs — which is the current behavior.
This commit is contained in:
parent
3298ddab3c
commit
e36771b57a
1 changed files with 9 additions and 1 deletions
|
|
@ -33,6 +33,7 @@ def filter_lfs_files(filenames: set[str]) -> None: # pragma: no cover (lfs)
|
||||||
def find_large_added_files(
|
def find_large_added_files(
|
||||||
filenames: Sequence[str],
|
filenames: Sequence[str],
|
||||||
maxkb: int,
|
maxkb: int,
|
||||||
|
allow_in_lfs: bool,
|
||||||
*,
|
*,
|
||||||
enforce_all: bool = False,
|
enforce_all: bool = False,
|
||||||
) -> int:
|
) -> int:
|
||||||
|
|
@ -40,7 +41,9 @@ def find_large_added_files(
|
||||||
# us about
|
# us about
|
||||||
retv = 0
|
retv = 0
|
||||||
filenames_filtered = set(filenames)
|
filenames_filtered = set(filenames)
|
||||||
filter_lfs_files(filenames_filtered)
|
|
||||||
|
if allow_in_lfs:
|
||||||
|
filter_lfs_files(filenames_filtered)
|
||||||
|
|
||||||
if not enforce_all:
|
if not enforce_all:
|
||||||
filenames_filtered &= added_files()
|
filenames_filtered &= added_files()
|
||||||
|
|
@ -64,6 +67,10 @@ def main(argv: Sequence[str] | None = None) -> int:
|
||||||
'--enforce-all', action='store_true',
|
'--enforce-all', action='store_true',
|
||||||
help='Enforce all files are checked, not just staged files.',
|
help='Enforce all files are checked, not just staged files.',
|
||||||
)
|
)
|
||||||
|
parser.add_argument(
|
||||||
|
'--allow-in-lfs', type=bool, default=true,
|
||||||
|
help='Whether to allow large files if stored in git lfs',
|
||||||
|
)
|
||||||
parser.add_argument(
|
parser.add_argument(
|
||||||
'--maxkb', type=int, default=500,
|
'--maxkb', type=int, default=500,
|
||||||
help='Maximum allowable KB for added files',
|
help='Maximum allowable KB for added files',
|
||||||
|
|
@ -73,6 +80,7 @@ def main(argv: Sequence[str] | None = None) -> int:
|
||||||
return find_large_added_files(
|
return find_large_added_files(
|
||||||
args.filenames,
|
args.filenames,
|
||||||
args.maxkb,
|
args.maxkb,
|
||||||
|
args.allow_in_lfs,
|
||||||
enforce_all=args.enforce_all,
|
enforce_all=args.enforce_all,
|
||||||
)
|
)
|
||||||
|
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue