Skip to content

Use pnpm exec in lefthook#693

Merged
romeerez merged 1 commit into
romeerez:mainfrom
IlyaSemenov:lefthook-use-npx
May 18, 2026
Merged

Use pnpm exec in lefthook#693
romeerez merged 1 commit into
romeerez:mainfrom
IlyaSemenov:lefthook-use-npx

Conversation

@IlyaSemenov
Copy link
Copy Markdown
Contributor

Currently, lefthook runs global oxlint and oxfmt. This is incorrect, it should run project-local binaries.

Without the fix:

% git ci -am "Export QuerySchema from pqb and orchid-orm"
╭───────────────────────────────────────╮
│ 🥊 lefthook v1.7.18  hook: pre-commit │
╰───────────────────────────────────────╯
┃  fmt ❯

sh: oxfmt: command not found

┃  lint ❯

sh: oxlint: command not found

with the fix:

sync hooks: ✔️ (pre-commit)
┃  fmt ❯

Finished in 95ms on 2 files using 12 threads.

┃  lint ❯

Found 0 warnings and 0 errors.
Finished in 8ms on 1 file with 65 rules using 12 threads.

@netlify
Copy link
Copy Markdown

netlify Bot commented May 18, 2026

Deploy Preview for orchid-orm canceled.

Name Link
🔨 Latest commit 9e7128d
🔍 Latest deploy log https://app.netlify.com/projects/orchid-orm/deploys/6a0aa5a4b80dc500082b89b9

@romeerez romeerez merged commit b07b0df into romeerez:main May 18, 2026
6 checks passed
@romeerez
Copy link
Copy Markdown
Owner

Thank you! I noticed it was failing for AI, but was postponing figuring out why, so here is the fix.

@IlyaSemenov IlyaSemenov deleted the lefthook-use-npx branch May 19, 2026 01:22
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.

2 participants