Skip to content

Exposes configPath to the run wrapper #564

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

Merged
merged 1 commit into from
Mar 1, 2025

Conversation

willfish
Copy link
Contributor

@willfish willfish commented Mar 1, 2025

What

  • Adds configPath to the run wrapper arguments as well as the module configuration

Why

  • I was just testing this on my machine and the configPath option wasn't available when calling the run wrapper

The following snippet of configuration is now working for me with this change

        pre-commit-check = pre-commit-hooks.lib.${system}.run {
          src = ./.;
          configPath = ".pre-commit-config-nix.yaml";
          hooks = {
            end-of-file-fixer.enable = true;
            flake-checker.enable = true;
            nil.enable = true;
            trim-trailing-whitespace.enable = true;
            trufflehog.enable = true;
          };
        };

See https://github.com/willfish/nix/blob/master/flake.nix for working implementation against my fork of this flake

@willfish
Copy link
Contributor Author

willfish commented Mar 1, 2025

This is just an extension of the following PR #563

@sandydoo sandydoo merged commit 25d4946 into cachix:master Mar 1, 2025
6 checks passed
@sandydoo
Copy link
Member

sandydoo commented Mar 1, 2025

Thanks!

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

Successfully merging this pull request may close these issues.

2 participants