Skip to content

chore: add lefthook git hooks and just task runner - #201

Merged
Erlend Ellefsen (erlendellefsen) merged 2 commits into
mainfrom
feat/just-lefthook
Aug 21, 2026
Merged

chore: add lefthook git hooks and just task runner#201
Erlend Ellefsen (erlendellefsen) merged 2 commits into
mainfrom
feat/just-lefthook

Conversation

@erlendellefsen

@erlendellefsen Erlend Ellefsen (erlendellefsen) commented Aug 21, 2026

Copy link
Copy Markdown
Collaborator

Summary

This change adds two developer tools to the repository.

  1. Lefthook manages git hooks. A pre-commit hook checks C# and TypeScript formatting and lint rules. A pre-push hook builds the .NET solution and type-checks the TypeScript client.
  2. Just is a task runner. The justfile adds recipes for setup, formatting, checks, and tests. It replaces long manual command sequences with short recipe names.

mise.toml now pins Deno, just, and lefthook, so mise install sets up all tools at once. docs/contributing.md explains the new setup step and lists the daily commands.

CI stays the source of truth for full test suites. The hooks catch common issues early, before a push.

- Add .lefthook.yml: format/lint checks on pre-commit, build/type-check on pre-push
- Add justfile: setup, format, check, test, test-contract, test-all, clean recipes
- Pin deno, just, lefthook in mise.toml
- Update docs/contributing.md with new setup and daily commands
- Ignore local lefthook override files in .gitignore
- Add a comment block that explains the required PR title format.
- Add a "How to test" section.
- Replace "Related issues" with a closes-issue note.
@erlendellefsen Erlend Ellefsen (erlendellefsen) changed the title Add lefthook git hooks and just task runner chore: add lefthook git hooks and just task runner Aug 21, 2026
@erlendellefsen
Erlend Ellefsen (erlendellefsen) merged commit 5adc493 into main Aug 21, 2026
15 of 16 checks passed
@erlendellefsen
Erlend Ellefsen (erlendellefsen) deleted the feat/just-lefthook branch August 21, 2026 13:27
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.

1 participant