psa-systems/lets-chat
Watch
9
0
Fork
You've already forked lets-chat
 0
Code Issues Pull requests Projects Releases 2 Packages Wiki Activity Actions

chore: add install-hooks + pre-commit recipes #67

Merged
David merged 2 commits from chore/pre-commit-hook into main 2026-05-11 14:28:01 +02:00
Conversation 0 Commits 2 Files changed 1 +14
David commented 2026-05-11 14:27:37 +02:00
Owner
Copy link

just install-hooks writes a stub at .git/hooks/pre-commit that execs just pre-commit. The new recipe depends on the existing check + test recipes, both of which already shell out to ./dev/cargo (Docker-based cargo wrapper using rust:1.88-slim-bookworm with persistent cargo + target volumes), so the pre-commit checks reuse the same dockerised toolchain that the rest of the dev workflow uses.

check.yml is left untouched - it already exists and currently references --features server, which does not match the workspace's standalone/saas features and may need a separate cleanup PR. The local just check recipe is the source of truth for what the pre-commit hook validates.

See governance/PRE_COMMIT.md for the org-wide standard.

Co-Authored-By: Claude Opus 4.7 (1M context) noreply@anthropic.com

`just install-hooks` writes a stub at .git/hooks/pre-commit that execs `just pre-commit`. The new recipe depends on the existing `check` + `test` recipes, both of which already shell out to `./dev/cargo` (Docker-based cargo wrapper using rust:1.88-slim-bookworm with persistent cargo + target volumes), so the pre-commit checks reuse the same dockerised toolchain that the rest of the dev workflow uses. `check.yml` is left untouched - it already exists and currently references `--features server`, which does not match the workspace's `standalone`/`saas` features and may need a separate cleanup PR. The local `just check` recipe is the source of truth for what the pre-commit hook validates. See governance/PRE_COMMIT.md for the org-wide standard. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
chore: add install-hooks + pre-commit recipes
Some checks failed
Check / clippy + fmt + tests (pull_request) Has been cancelled
Details
1fd02b76d1 
`just install-hooks` writes a stub at .git/hooks/pre-commit that execs `just pre-commit`. The new recipe depends on the existing `check` + `test` recipes, both of which already shell out to `./dev/cargo` (Docker-based cargo wrapper using rust:1.88-slim-bookworm with persistent cargo + target volumes), so the pre-commit checks reuse the same dockerised toolchain that the rest of the dev workflow uses.

`check.yml` is left untouched - it already exists and currently references `--features server`, which does not match the workspace's `standalone`/`saas` features and may need a separate cleanup PR. The local `just check` recipe is the source of truth for what the pre-commit hook validates.

See governance/PRE_COMMIT.md for the org-wide standard.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Merge branch 'main' into chore/pre-commit-hook
Some checks failed
Check / clippy + fmt + tests (pull_request) Failing after 2m29s
Details
6d89711216
David merged commit ccc4c9990d into main 2026-05-11 14:28:01 +02:00
David deleted branch chore/pre-commit-hook 2026-05-11 14:28:01 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
psa-systems/lets-chat!67
No description provided.