Skip to content

Remove stale Claude pre-commit hook#124

Merged
ANcpLua merged 1 commit into
mainfrom
dev/test-claude-review-analyzers-hook-20260507065919
May 7, 2026
Merged

Remove stale Claude pre-commit hook#124
ANcpLua merged 1 commit into
mainfrom
dev/test-claude-review-analyzers-hook-20260507065919

Conversation

@ANcpLua
Copy link
Copy Markdown
Owner

@ANcpLua ANcpLua commented May 7, 2026

What

  • Delete the tracked .claude/hooks/pre-commit-guard.sh local hook.
  • Remove the matching unreleased changelog bullet.

Why

The repository should not carry Claude-local commit/stash/push guard logic as project source. Removing the changelog bullet keeps the unreleased notes honest.

Validation

Not run locally. This PR is intentionally small and ready so GitHub PR workflows, including Claude Code Review, trigger on a real cleanup change.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 7, 2026

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (2)
  • .claude/hooks/pre-commit-guard.sh is excluded by none and included by none
  • CHANGELOG.md is excluded by none and included by none

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 10fd681a-5457-48eb-9db3-922f72de7866

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 7, 2026

@coderabbitai autofix

@ghost ghost enabled auto-merge (squash) May 7, 2026 05:00
@claude
Copy link
Copy Markdown

claude Bot commented May 7, 2026

Claude encountered an error —— View job


I'll analyze this and get back to you.

@ANcpLua ANcpLua merged commit a3c2129 into main May 7, 2026
13 checks passed
@ANcpLua ANcpLua deleted the dev/test-claude-review-analyzers-hook-20260507065919 branch May 7, 2026 05:07
Copilot AI review requested due to automatic review settings May 7, 2026 06:18
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 7, 2026

Note

Autofix is a beta feature. Expect some limitations and changes as we gather feedback and continue to improve it.

Autofix skipped. No unresolved CodeRabbit review comments with fix instructions found.

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