git-hooks.nix icon indicating copy to clipboard operation
git-hooks.nix copied to clipboard

fix: add configFile option

Open diefans opened this issue 1 year ago • 1 comments

This fixes #345, by adding a configFile option to change the default (.pre-commit-config.yaml) and so enables the parallel usage of devenv and non-devenv maintained pre-commit hooks.

diefans avatar Sep 26 '24 07:09 diefans

  1. There are a few more pre-commit calls in run that would need to be adjusted. https://github.com/cachix/git-hooks.nix/blob/6d7586d0d35ccd1e87bb0ffb7c0d38b4cc0fe469/modules/pre-commit.nix#L55-L83
  2. I'm not sure about the name of the option. We may also need to expose configFile itself in the future, e.g. #494. Maybe configPath would be better?

sandydoo avatar Oct 11 '24 08:10 sandydoo

@sandydoo ...I am full of work at the moment, but this topic is not forgotten!

diefans avatar Jan 15 '25 09:01 diefans

This is a make or break option for me at work since I work with pre-existing pre-commit configuration and I can't just delete/.gitignore that. I'm using flakes in about 10 repos that I can just add this tooling for and make my life a bunch easier

willfish avatar Feb 22 '25 20:02 willfish