Add pass_filenames support

Author: mxr

pre_commit_mirror_maker/all/.pre-commit-hooks.yaml

@@ -5,6 +5,7 @@
     language: {language}
     '{match_key}': {match_val}
     args: {args}
+    pass_filenames: {pass_filenames}
     require_serial: {require_serial}
     additional_dependencies: {additional_dependencies}
     minimum_pre_commit_version: {minimum_pre_commit_version!r}

pre_commit_mirror_maker/main.py

@@ -73,6 +73,11 @@ def main(argv: Sequence[str] | None = None) -> int:
         '--require-serial', action='store_true',
         help='Set `require_serial: true` for the hook',
     )
+    parser.add_argument(
+        '--pass-filenames', action=argparse.BooleanOptionalAction,
+        default=True,
+        help='Set `pass_filenames` for the hook',
+    )
     args = parser.parse_args(argv)
 
     minimum_pre_commit_version = '0'
@@ -105,6 +110,7 @@ def main(argv: Sequence[str] | None = None) -> int:
         match_key=match_key,
         match_val=match_val,
         args=json.dumps(split_by_commas(args.args)),
+        pass_filenames=json.dumps(args.pass_filenames),
         require_serial=json.dumps(args.require_serial),
         minimum_pre_commit_version=minimum_pre_commit_version,
     )

pre_commit_mirror_maker/make_repo.py

@@ -45,6 +45,7 @@ def _commit_version(
         version: str,
         **fmt_vars: str,
 ) -> None:
+    fmt_vars.setdefault('pass_filenames', 'true')
     # 'all' writes the .version and .pre-commit-hooks.yaml files
     files = importlib.resources.files('pre_commit_mirror_maker')
     with importlib.resources.as_file(files) as files_p:

tests/main_test.py

@@ -43,7 +43,8 @@ def test_main_passes_args(mock_make_repo):
         language='ruby', name='scss-lint', description='',
         entry='scss-lint-entry',
         id='scss-lint-id', match_key='files', match_val=r'\.scss$', args='[]',
-        require_serial='false', minimum_pre_commit_version='0',
+        pass_filenames='true', require_serial='false',
+        minimum_pre_commit_version='0',
     )
 
 
@@ -80,6 +81,17 @@ def test_main_with_args(mock_make_repo):
     assert mock_make_repo.call_args[1]['args'] == expected
 
 
+def test_main_no_pass_filenames(mock_make_repo):
+    assert not main.main((
+        '.',
+        '--language', 'python',
+        '--package-name', 'yapf',
+        '--files-regex', r'\.py$',
+        '--no-pass-filenames',
+    ))
+    assert mock_make_repo.call_args[1]['pass_filenames'] == 'false'
+
+
 def test_main_with_types_and_types_or(mock_make_repo, capsys):
     with pytest.raises(SystemExit) as exc_info:
         main.main((

tests/make_repo_test.py

@@ -76,8 +76,8 @@ def test_arguments(in_git_dir):
         version='0.6.2', language='python', name='yapf',
         description='Yet another Python formatter.', entry='yapf', id='yapf',
         match_key='files', match_val=r'\.py$', args='["-i"]',
-        additional_dependencies='["scikit-learn"]', require_serial='false',
-        minimum_pre_commit_version='0',
+        pass_filenames='false', additional_dependencies='["scikit-learn"]',
+        require_serial='false', minimum_pre_commit_version='0',
     )
     contents = in_git_dir.join('.pre-commit-hooks.yaml').read()
     assert yaml.safe_load(contents) == [{
@@ -88,6 +88,7 @@ def test_arguments(in_git_dir):
         'language': 'python',
         'files': r'\.py$',
         'args': ['-i'],
+        'pass_filenames': False,
         'require_serial': False,
         'additional_dependencies': ['scikit-learn'],
         'minimum_pre_commit_version': '0',