add-pre-commit #1413

fricklerhandwerk · 2024-05-22T21:34:53Z

fricklerhandwerk commented

2024-05-22 21:34:53 +00:00

#1369 had to be reverted in #1408 due to a build failure in the git-hooks helper, which is now fixed upstream.

https://git.clan.lol/clan/clan-core/pulls/1369 had to be reverted in https://git.clan.lol/clan/clan-core/pulls/1408 due to a build failure in the git-hooks helper, which is now fixed upstream.

fricklerhandwerk added 1 commit 2024-05-22 21:34:53 +00:00

add pre-commit check

checks / checks-impure (pull_request) Successful in 2m30s

Details

f164cb703c

make sure things are sane before they hit CI.
this re-purposes the existing treefmt configuration.

this adds a custom installer for pre-commit hooks, which is inspired by
pre-commit.nix[0], but is much more minimal than the underlying
pre-commit[1] and builds on a historic idea[2] from this repository.

[0]: https://github.com/cachix/git-hooks.nix
[1]: https://github.com/pre-commit/pre-commit
[2]: 930923512c

fricklerhandwerk force-pushed add-pre-commit from f164cb703c to 96d421c308

2024-05-22 21:35:24 +00:00

Compare

Qubasa merged commit d7ef88c67c into main

2024-05-24 12:39:56 +00:00

Qubasa referenced this issue from a commit

2024-05-24 12:39:58 +00:00

Merge pull request 'add-pre-commit' (#1413) from fricklerhandwerk/clan-core:add-pre-commit into main

Sign in to join this conversation.

No reviewers