Skip to content

docs: correct Lefthook commit message configuration - #1499

Merged
mrexox merged 1 commit into
evilmartians:masterfrom
codersjj:patch-1
Aug 21, 2026
Merged

mrexox merged 1 commit into
evilmartians:masterfrom
codersjj:patch-1

Conversation

@codersjj

Copy link
Copy Markdown

Closes # (issue)

Context

Incorrect type. Expected "string".

Changes

Updated LEFTHOOK environment variable to be a string.

Updated LEFTHOOK environment variable to be a string.
@codersjj
codersjj requested a review from mrexox as a code owner August 20, 2026 02:49
@greptile-apps

greptile-apps Bot commented Aug 20, 2026

Copy link
Copy Markdown

Confidence Score: 4/5

The PR is safe to merge, although the corresponding runnable commitlint example should be updated to avoid retaining conflicting guidance.

The changed Markdown now uses the schema-required string value, but users consulting the matching runnable example will still encounter the numeric form this PR intends to correct.

Files Needing Attention: docs/examples/commitlint.md and examples/commitlint/lefthook.yml

Reviews (1): Last reviewed commit: "fix: correct Lefthook commit message con..." | Re-trigger Greptile

run: yarn run cz --hook # Or npx cz --hook
env:
LEFTHOOK: 0
LEFTHOOK: "0"

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Commitlint examples remain inconsistent

This correction applies only to the Markdown snippet, while examples/commitlint/lefthook.yml still uses the schema-invalid numeric value LEFTHOOK: 0; users consulting the runnable example therefore continue to receive the conflicting configuration this change is intended to correct.

@mrexox mrexox changed the title fix: correct Lefthook commit message configuration docs: correct Lefthook commit message configuration Aug 21, 2026
@mrexox
mrexox merged commit db7ca05 into evilmartians:master Aug 21, 2026
12 checks passed
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