Skip to content

chore: apply exec permission to shell hook script - #1054

Merged
tbouffard merged 1 commit into
mainfrom
chore/apply_exec_permission_to_all_scripts
May 4, 2026
Merged

chore: apply exec permission to shell hook script#1054
tbouffard merged 1 commit into
mainfrom
chore/apply_exec_permission_to_all_scripts

Conversation

@redfish4ktc

Copy link
Copy Markdown
Member

Mark .claude/hooks/block-build.sh as executable in the git index so it can run as a Claude Code PreToolUse hook
without requiring users to chmod after cloning. The script has a #!/bin/bash shebang and is invoked directly by
the harness, which fails silently if the bit is missing.

Audited all tracked .sh/.bash files in the repo, plus every tracked file with a #!/bin/bash or #!/usr/... shebang;
this hook was the only one missing the executable bit.

Mark .claude/hooks/block-build.sh as executable in the git index so it can run as a Claude Code PreToolUse hook
without requiring users to chmod after cloning. The script has a #!/bin/bash shebang and is invoked directly by
the harness, which fails silently if the bit is missing.

Audited all tracked .sh/.bash files in the repo, plus every tracked file with a #!/bin/bash or #!/usr/... shebang;
this hook was the only one missing the executable bit.
@redfish4ktc redfish4ktc added the chore Build, CI/CD or repository tasks (issues/PR maintenance, environments, ...) label May 3, 2026
@coderabbitai

coderabbitai Bot commented May 3, 2026

Copy link
Copy Markdown

Important

Review skipped

Review was skipped as selected files did not have any reviewable changes.

💤 Files selected but had no reviewable changes (1)
  • .claude/hooks/block-build.sh
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 4092491a-d3be-469a-a18c-3c90c90fd5fa

📥 Commits

Reviewing files that changed from the base of the PR and between e34e566 and 7aa2c40.

📒 Files selected for processing (1)
  • .claude/hooks/block-build.sh

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

Tip

💬 Introducing Slack Agent: The best way for teams to turn conversations into code.

Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.

  • Generate code and open pull requests
  • Plan features and break down work
  • Investigate incidents and troubleshoot customer tickets together
  • Automate recurring tasks and respond to alerts with triggers
  • Summarize progress and report instantly

Built for teams:

  • Shared memory across your entire org—no repeating context
  • Per-thread sandboxes to safely plan and execute work
  • Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get started


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share
Review rate limit: 7/8 reviews remaining, refill in 7 minutes and 30 seconds.

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

@sonarqubecloud

sonarqubecloud Bot commented May 3, 2026

Copy link
Copy Markdown

@tbouffard
tbouffard merged commit 1a0c859 into main May 4, 2026
6 checks passed
@tbouffard
tbouffard deleted the chore/apply_exec_permission_to_all_scripts branch May 4, 2026 08:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore Build, CI/CD or repository tasks (issues/PR maintenance, environments, ...)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants