Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Config file not supported? #360

Open
BenLeadbetter opened this issue Dec 3, 2023 · 1 comment
Open

Config file not supported? #360

BenLeadbetter opened this issue Dec 3, 2023 · 1 comment

Comments

@BenLeadbetter
Copy link

Hi, nice tool!

I had a poke around and found the --config and --dump-config arguments to the committed exec. I tried out the following:

committed --dump-config .committed-config

And set my .pre-commit-config to read from this file:

- repo: https://github.com/crate-ci/committed
  rev: v1.0.20
  hooks:
  - id: committed
    args: [--config, .committed-config]

However, this makes the hook fail with the following error:

failed to parse revision specifier - Invalid pattern '.git/COMMIT_EDITMSG'; class=Invalid (3); code=InvalidSpec (-12)

Is this workflow not supported? Is there another way to specify configuration via my .pre-commit-config file? From the output of --dump-config, it looks like there's some nice configuration options there which I would love to try out.

Thanks

@epage
Copy link
Collaborator

epage commented Dec 4, 2023

Running directly, it works.

Something must be going wrong with pre-commit but unsure what.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants