Skip to content

Commit da14a37

Browse files
committed
Update Claude workflow and documentation for commit message guidelines
1 parent a38356b commit da14a37

File tree

3 files changed

+18
-102
lines changed

3 files changed

+18
-102
lines changed

.github/workflows/claude-code-review.yml

Lines changed: 0 additions & 78 deletions
This file was deleted.

.github/workflows/claude.yml

Lines changed: 16 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -19,9 +19,9 @@ jobs:
1919
(github.event_name == 'issues' && (contains(github.event.issue.body, '@claude') || contains(github.event.issue.title, '@claude')))
2020
runs-on: ubuntu-latest
2121
permissions:
22-
contents: read
23-
pull-requests: read
24-
issues: read
22+
contents: write
23+
pull-requests: write
24+
issues: write
2525
id-token: write
2626
actions: read # Required for Claude to read CI results on PRs
2727
steps:
@@ -39,26 +39,18 @@ jobs:
3939
# This is an optional setting that allows Claude to read CI results on PRs
4040
additional_permissions: |
4141
actions: read
42-
43-
# Optional: Specify model (defaults to Claude Sonnet 4, uncomment for Claude Opus 4)
44-
# model: "claude-opus-4-20250514"
45-
46-
# Optional: Customize the trigger phrase (default: @claude)
47-
# trigger_phrase: "/claude"
48-
49-
# Optional: Trigger when specific user is assigned to an issue
50-
# assignee_trigger: "claude-bot"
51-
52-
# Optional: Allow Claude to run specific commands
53-
# allowed_tools: "Bash(npm install),Bash(npm run build),Bash(npm run test:*),Bash(npm run lint:*)"
54-
42+
43+
model: "claude-opus-4-20250514"
44+
45+
allowed_tools: "Bash(go:*),Bash(golangci-lint:*),Bash(make:*),View,GlobTool,GrepTool,BatchTool,mcp__github"
46+
5547
# Optional: Add custom instructions for Claude to customize its behavior for your project
56-
# custom_instructions: |
57-
# Follow our coding standards
58-
# Ensure all new code has tests
59-
# Use TypeScript for new files
60-
61-
# Optional: Custom environment variables for Claude
62-
# claude_env: |
63-
# NODE_ENV: test
48+
custom_instructions: |
49+
Follow our coding standards
50+
Ensure tests and lints pass before making commits or PRs
51+
Run 'make test' to verify all tests pass
52+
Use Go conventions and best practices
53+
When reviewing code: provide brief summary of changes,
54+
list concerns as bullets (keep short and understandable),
55+
end with approval status using red/green icons for visibility.
6456

CLAUDE.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -88,6 +88,8 @@ The application uses TOML configuration files. See `config.toml.example` for all
8888
### Git Workflow
8989
- Write short, expressive commit messages that clearly describe the change
9090
- Do not include automated signatures or generation notices in commit messages
91+
- Don't add "Generated with Claude Code" to commit messages
92+
- Don't add "Co-Authored-By: Claude [email protected]" to commit messages
9193
- Keep commits focused and atomic - one logical change per commit
9294
- Ensure the build passes before pushing commits
9395

0 commit comments

Comments
 (0)