Skip to content

feat(nix): Add pre-commit hook to check flake input follows#42

Merged
tlater-famedly merged 1 commit into
mainfrom
tlater/flake-follows
Jul 22, 2026
Merged

feat(nix): Add pre-commit hook to check flake input follows#42
tlater-famedly merged 1 commit into
mainfrom
tlater/flake-follows

Conversation

@tlater-famedly

Copy link
Copy Markdown
Contributor

No description provided.

@cursor

cursor Bot commented Jul 14, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
Developer tooling only; no runtime app, auth, or data-path changes.

Overview
Adds a flake-follows local pre-commit hook that runs flake-edit follow on flake.nix and flake.lock so flake inputs stay recursively de-duplicated via follows.

The hook is declared in .pre-commit-config.yaml and wired into the Nix standards flake via new nix/nix/pre-commit-hooks.nix, which registers it on the root workspace and adds pkgs.flake-edit to prek-pre-commit.package.runtimePkgs. nix/nix/default.nix now imports that module alongside formatting.

Reviewed by Cursor Bugbot for commit 3c33b0b. Bugbot is set up for automated code reviews on this repo. Configure here.

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.

Fix All in Cursor

Bugbot Autofix is ON. A cloud agent has been kicked off to fix the reported issue. You can view the agent here.

Reviewed by Cursor Bugbot for commit 3c33b0b. Configure here.

Comment thread nix/nix/pre-commit-hooks.nix
@tlater-famedly
tlater-famedly force-pushed the tlater/flake-follows branch 2 times, most recently from 9a97aa3 to 921a2a8 Compare July 21, 2026 10:04
@tlater-famedly
tlater-famedly changed the base branch from main to tlater/fix-treefmt-prek July 21, 2026 10:05
@tlater-famedly
tlater-famedly force-pushed the tlater/fix-treefmt-prek branch from 483acc5 to 72fba97 Compare July 21, 2026 10:06
Base automatically changed from tlater/fix-treefmt-prek to main July 21, 2026 10:09

@RasmusRendal RasmusRendal left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works!

@tlater-famedly
tlater-famedly merged commit 26b385a into main Jul 22, 2026
2 checks passed
@tlater-famedly
tlater-famedly deleted the tlater/flake-follows branch July 22, 2026 09:43
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