Skip to content

build: fix hooks verification#19

Merged
longlho merged 1 commit intomainfrom
01-05-build_fix_hooks_verification
Jan 6, 2026
Merged

build: fix hooks verification#19
longlho merged 1 commit intomainfrom
01-05-build_fix_hooks_verification

Conversation

@longlho
Copy link
Copy Markdown
Member

@longlho longlho commented Jan 6, 2026

TL;DR

Replace commitizen with commitlint for commit message validation and typos with cspell for spell checking.

What changed?

  • Added commitlint configuration with commitlint.config.js using conventional commit standards
  • Replaced typos with cspell for spell checking in the pre-commit hook
  • Updated lefthook.yml to use cspell instead of typos and commitlint instead of commitizen
  • Added new dev dependencies: @commitlint/cli, @commitlint/config-conventional, and cspell
  • Simplified the oxfmt command in lefthook by removing the exec prefix
  • Removed empty .prettierignore file

How to test?

  1. Make a commit with an invalid commit message format to verify commitlint rejects it
  2. Make a commit with a valid conventional commit message to verify it passes
  3. Introduce a spelling error in a file and verify cspell catches it in the pre-commit hook

Why make this change?

This change standardizes the commit message format using the widely-adopted conventional commits specification, which improves commit history readability and enables automated changelog generation. It also replaces the optional typos tool with cspell, which provides more consistent spell checking across the team without requiring external tool installation.

Copy link
Copy Markdown
Member Author

longlho commented Jan 6, 2026

This stack of pull requests is managed by Graphite. Learn more about stacking.

@longlho longlho force-pushed the 01-05-build_fix_hooks_verification branch from e651f3f to 68b454c Compare January 6, 2026 03:21
@longlho longlho marked this pull request as ready for review January 6, 2026 03:22
@longlho longlho merged commit c0a686d into main Jan 6, 2026
10 checks passed
Copy link
Copy Markdown
Member Author

longlho commented Jan 6, 2026

Merge activity

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