Skip to content

feat: add block-no-verify PreToolUse hook to .claude/settings.json#13641

Closed
tupe12334 wants to merge 1 commit intomapbox:mainfrom
tupe12334:add-block-no-verify
Closed

feat: add block-no-verify PreToolUse hook to .claude/settings.json#13641
tupe12334 wants to merge 1 commit intomapbox:mainfrom
tupe12334:add-block-no-verify

Conversation

@tupe12334
Copy link
Copy Markdown

Summary\n\nAdds block-no-verify@1.1.2 as a PreToolUse Bash hook in .claude/settings.json.\n\nCloses #13640\n\n---\n_Disclosure: I am the author and maintainer of block-no-verify._

Prevents agents from bypassing git hooks via the hook-skip flag.
@tupe12334 tupe12334 requested a review from a team as a code owner March 19, 2026 20:56
@tupe12334 tupe12334 requested review from mourner and removed request for a team March 19, 2026 20:56
@github-actions
Copy link
Copy Markdown

Hey, @tupe12334 👋 Thanks for your contribution to Mapbox GL JS!

Important: This repository does not accept direct merges. All changes go through our internal review process.

What happens next:

  1. A team member will review your PR here first
  2. If it looks good, they will import it to our internal repository for further review
  3. If approved, changes will be synced back here via our release process

Please respond to any review comments on this PR. For more details, see CONTRIBUTING.md.

@mourner
Copy link
Copy Markdown
Member

mourner commented Mar 19, 2026

Nah, thanks

@mourner mourner closed this Mar 19, 2026
@tupe12334 tupe12334 deleted the add-block-no-verify branch March 22, 2026 22:39
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