Skip to content

fix(commit-msg): reject whitespace-only descriptions - #13

Merged
taigrr merged 1 commit into
masterfrom
cd/round7-20261108
Nov 9, 2026
Merged

taigrr merged 1 commit into
masterfrom
cd/round7-20261108

Conversation

@taigrr

@taigrr taigrr commented Nov 8, 2026

Copy link
Copy Markdown
Owner

Summary

  • Require at least one non-whitespace character in conventional commit descriptions.
  • Cover whitespace-only descriptions with scopes and breaking markers, while preserving meaningful descriptions with surrounding spaces or tabs.
  • Document the description requirement. Merge and fixup/squash exemptions are unchanged.

Validation

  • Regression rejects the original implementation (whitespace-only scoped description incorrectly exits successfully).
  • All four shell regression suites pass.
  • ShellCheck passes for all hooks, install/uninstall scripts, and tests.
  • git diff --check passes.

Independent of pending staged-file-size PR #12; no shared files changed.

@taigrr
taigrr merged commit fa6382e into master Nov 9, 2026
1 check 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.

1 participant