Skip to content

fix(release): run commit hooks when publishing - #1463

Merged
Chemaclass merged 1 commit into
mainfrom
fix/release-respect-hooks
Nov 7, 2026
Merged

Chemaclass merged 1 commit into
mainfrom
fix/release-respect-hooks

Conversation

@Chemaclass

Copy link
Copy Markdown
Member

Background

Release commits skip Git hooks through git commit -n. Publishing should run the same checks as other commits.

Changes

  • Remove the hook bypass from publishing and sandbox release commits.
  • Verify that a rejecting pre-commit hook prevents both paths from committing.

Validation: 98 focused release tests (150 assertions), Bash syntax, ShellCheck, formatting, and static review passed.

Checklist

  • Changelog checked: no public runtime behavior changed.
  • Documentation checked: the existing no-hook-bypass rule describes the intended behavior.

@Chemaclass Chemaclass added the bug Something isn't working label Nov 7, 2026
@Chemaclass Chemaclass self-assigned this Nov 7, 2026
@Chemaclass
Chemaclass merged commit ee84c7b into main Nov 7, 2026
39 checks passed
@Chemaclass
Chemaclass deleted the fix/release-respect-hooks branch November 7, 2026 23:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant