No `rev` in `local` repo

[jace]

Release 0.0.2 is failing for me:

sync-pre-commit-deps.....................................................Failed
- hook id: sync-pre-commit-deps
- exit code: 1

Traceback (most recent call last):
  File "/Users/jace/.cache/pre-commit/repoluscuez_/py_env-python3/bin/sync-pre-commit-deps", line 8, in <module>
    sys.exit(main())
             ^^^^^^
  File "/Users/jace/.cache/pre-commit/repoluscuez_/py_env-python3/lib/python3.11/site-packages/sync_pre_commit_deps.py", line 60, in main
    cleaned_rev = repo['rev'].removeprefix('v')
                  ~~~~^^^^^^^
  File "/Users/jace/.cache/pre-commit/repoluscuez_/py_env-python3/lib/python3.11/site-packages/ruamel/yaml/comments.py", line 853, in __getitem__
    return ordereddict.__getitem__(self, key)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
KeyError: 'rev'

I have a local repo for mypy as it needs an environment with all deps installed:

  - repo: local
    hooks:
      - id: mypy
        name: mypy
        entry: .venv/bin/mypy

pre-commit will not let me add a fake rev tag to it:

An error has occurred: InvalidConfigError:
==> File .pre-commit-config.yaml
==> At Config()
==> At key: repos
==> At Repository(repo='local')
=====> Expected rev to be absent when repo is any of ('local', 'meta'), found rev: 'any'

This error is from the addition of mypy to the supported list in cbe3dd6. I have renamed it to mypy-local as a workaround, but the local repo should be entirely skipped.

[mxr]

Thanks for reporting. The fix is straightforward, I can work on that today.

I have a local repo for mypy as it needs an environment with all deps installed:

Not sure if you are aware but you don't need a local repo to do that - see https://github.com/pre-commit/mirrors-mypy/blob/bbc3dc1f890007061f18f17e2334f216ea9e5df7/README.md?plain=1#L31-L39 for how to pass dependencies to the non-local mypy hook

[jace]

Unfortunately, --ignore-missing-imports means the report via pre-commit is not the same as in the venv – errors will be missed. I lose mypy in pre-commit.ci by using a local repo, but can still run it from GitHub Actions.

Additional dependencies have to be kept in sync with requirements.txt/pyproject.toml/lockfile, and also validated for sync in a pre-commit hook, for which I've not found a solution.

[mxr]

For the first issue you could override args as documented in the README. For the second issue yes I agree - you would have to write a pre-commit hook (or some other solution) for that

[mxr]

The fix and a unit test are merged. I just released v0.0.3. Please give it a try. Thanks!