Skip to content

chore: speed up pre-commit hooks - #1661

Merged
danielroe merged 2 commits into
mainfrom
chore/precommit
Feb 26, 2026
Merged

chore: speed up pre-commit hooks#1661
danielroe merged 2 commits into
mainfrom
chore/precommit

Conversation

@danielroe

Copy link
Copy Markdown
Member

🔗 Linked issue

🧭 Context

📚 Description

@vercel

vercelBot commented Feb 26, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

ProjectDeploymentActionsUpdated (UTC)
npmx.devReadyReadyPreview, CommentFeb 26, 2026 10:04am
2 Skipped Deployments
ProjectDeploymentActionsUpdated (UTC)
docs.npmx.devIgnoredIgnoredPreviewFeb 26, 2026 10:04am
npmx-lunariaIgnoredIgnoredFeb 26, 2026 10:04am

Request Review

@codecov

codecovBot commented Feb 26, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

@coderabbitai

coderabbitaiBot commented Feb 26, 2026

Copy link
Copy Markdown
Contributor

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between c476ca9 and 8043488.

📒 Files selected for processing (1)
  • package.json

📝 Walkthrough

Walkthrough

The package.json was changed to call local binaries directly in hooks and lint-staged rules. The pre-commit hook now runs ./node_modules/.bin/lint-staged instead of npx lint-staged. The lint:css script was removed. lint-staged entries were updated to use ./node_modules/.bin/oxfmt and ./node_modules/.bin/oxlint --fix for formatting/linting, and *.vue now runs node scripts/unocss-checker.ts.

🚥 Pre-merge checks | ❌ 1

❌ Failed checks (1 warning)

Check nameStatusExplanationResolution
Description check⚠️ WarningThe PR description contains only the repository template with empty sections and no substantive content describing the actual changes or their rationale.Provide a detailed description explaining why the package.json commands were changed from package manager wrappers to direct binary paths, how this speeds up pre-commit hooks, and any relevant testing or compatibility considerations.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch chore/precommit

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@danielroe
danielroe merged commit 976070d into mainFeb 26, 2026
20 checks passed
@danielroe
danielroe deleted the chore/precommit branch February 26, 2026 10:48
Sign up for freeto 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

@danielroe