Skip to content

Onboarding#28

Open
JacobCoffee wants to merge 3 commits intomainfrom
onboarding
Open

Onboarding#28
JacobCoffee wants to merge 3 commits intomainfrom
onboarding

Conversation

@JacobCoffee
Copy link
Member

Address some missed items that help guide users on getting set up contributing here

  • run precommit hooks in ci so people dont need prek/precommit
  • ensure we run all checks in ci that we do in precommit
  • Add install docs for bun, prek on all platforms
  • Update make targets to check platform, and add helper message printsouts

@JacobCoffee JacobCoffee requested review from Copilot and hugovk March 3, 2026 18:58
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

## Python Insider Blog

Blog for Python core team, mostly for blogging about releases.
The official blog for the Python core teammostly release announcements.
Copy link
Member

Choose a reason for hiding this comment

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

But we want other stuff too :)

Suggested change
The official blog for the Python core team — mostly release announcements.
The official blog for the Python core team.

Comment on lines +30 to +37
# macOS
brew install prefix-dev/prek/prek

### Authors
# Linux / Windows (standalone installer)
curl -fsSL https://prek.j178.dev/install.sh | bash

# Or via cargo / pip / other methods
cargo install prek
Copy link
Member

Choose a reason for hiding this comment

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

All of these are wrong!

https://prek.j178.dev/installation/

Maybe we should just point to the install page and let people choose their preferred method? I expect many will prefer pip, uv or Homebrew.

make dev # starts dev server at http://127.0.0.1:4321
```

`make install` runs `bun install` (dependencies) and `prek install` (git hooks).
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
`make install` runs `bun install` (dependencies) and `prek install` (git hooks).
`make install` runs `bun install` (dependencies) and `prek install` (Git hooks).

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.

3 participants