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

Fixes xargs partition length on Windows for EXE hooks

Browse source
This commit is contained in:
Alex Marvin 2024-02-07 21:17:07 -06:00 • committed by GitHub
parent 73848383f2
commit 9da305e185
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 2 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

2
pre_commit/xargs.py
View file

@ -155,7 +155,7 @@ def xargs(
# on windows, batch files have a separate length limit than windows itself
if (
sys.platform == 'win32' and
cmd[0].lower().endswith(('.bat', '.cmd'))
cmd[0].lower().endswith(('.bat', '.cmd', '.exe'))
): # pragma: win32 cover
# this is implementation details but the command gets translated into
# full/path/to/cmd.exe /c *cmd

2
tests/xargs_test.py
View file

@ -243,7 +243,7 @@ def test_xargs_color_true_makes_tty():
@pytest.mark.xfail(os.name == 'posix', reason='nt only')
@pytest.mark.parametrize('filename', ('t.bat', 't.cmd', 'T.CMD'))
@pytest.mark.parametrize('filename', ('t.bat', 't.cmd', 'T.CMD', 't.exe'))
def test_xargs_with_batch_files(tmpdir, filename):
f = tmpdir.join(filename)
f.write('echo it works\n')