Skip to content

Onboarding #28

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
JacobCoffee wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Onboarding #28

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
1823b3c
docs: update gaps in getting started contributing
JacobCoffee Mar 3, 2026
10a0b57
ci: run all precommit things in ci so people dont need precommit
JacobCoffee Mar 3, 2026
7e061e7
use action to run hooks
JacobCoffee Mar 3, 2026
6eae61f
do not require prek
JacobCoffee Mar 3, 2026
4989813
check for bun install on all platforms
JacobCoffee Mar 3, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
do not require prek
  • Loading branch information
@JacobCoffee
JacobCoffee committed Mar 3, 2026
commit 6eae61ff3e780c43c60d6af2b41215079a18374d
6 changes: 5 additions & 1 deletion Makefile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,11 @@ help: ## Show this help message
.PHONY: install
install: ## Install dependencies and git hooks
bun install
prek install
@if command -v prek >/dev/null 2>&1; then \
prek install; \
else \
echo "prek not found — skipping git hook setup (optional, see README)"; \
fi

## Development

Expand Down