Skip to content

git-hooks: change default package to pkgs.prek#2304

Merged
domenkozar merged 1 commit into
mainfrom
git-hooks-prek
Feb 2, 2026
Merged

git-hooks: change default package to pkgs.prek#2304
domenkozar merged 1 commit into
mainfrom
git-hooks-prek

Conversation

@domenkozar

@domenkozar domenkozar commented Nov 24, 2025

Copy link
Copy Markdown
Member

Add a changelog since it's a breaking change #2300

Upstream: cachix/git-hooks.nix#656

@github-actions

github-actions Bot commented Nov 24, 2025

Copy link
Copy Markdown
Contributor

🔍 Suggested Reviewers

Based on git blame analysis of the changed lines, the following contributors have significant experience with the modified code:

  • @sandydoo - 100.0% of changed lines (1 lines)

Please consider reviewing this PR as you have authored significant portions of the code being modified. Your expertise would be valuable! 🙏

This comment was automatically generated by git-blame-auto-reviewer

Last updated: 2026-02-02T10:17:37.520Z

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Nov 24, 2025

Copy link
Copy Markdown

Deploying devenv with  Cloudflare Pages  Cloudflare Pages

Latest commit: 44c6025
Status:⚡️  Build in progress...

View logs

@compactcode

Copy link
Copy Markdown

Just learning about prek, looks like a nice upgrade. Would this require an update to the Claude Code integration as well?

cd "$DEVENV_ROOT" && pre-commit run

@domenkozar

Copy link
Copy Markdown
Member Author

Just learning about prek, looks like a nice upgrade. Would this require an update to the Claude Code integration as well?

cd "$DEVENV_ROOT" && pre-commit run

Good catch, thanks, fixed!

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@domenkozar
domenkozar merged commit f33d92e into main Feb 2, 2026
12 of 15 checks passed
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