Skip to content

Commit d381c3b

Browse files
authored
More tweaks (#1162)
1 parent 7f6cde6 commit d381c3b

File tree

1 file changed

+10
-6
lines changed

1 file changed

+10
-6
lines changed

.github/workflows/claude.yml

Lines changed: 10 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -5,12 +5,10 @@ name: Claude PR Comment Review
55
on:
66
issue_comment:
77
types: [created]
8-
pull_request_review_comment:
9-
types: [created]
108

119
jobs:
1210
review:
13-
if: ${{ contains(github.event.comment.body, '@claude') && (github.event.issue.pull_request != null || github.event.pull_request != null) }}
11+
if: ${{ contains(github.event.comment.body, '@claude') && github.event.issue.pull_request != null }}
1412
runs-on: ubuntu-latest
1513
permissions:
1614
contents: read
@@ -21,23 +19,29 @@ jobs:
2119
with:
2220
fetch-depth: 1
2321

22+
- run: find
23+
2424
- uses: anthropics/claude-code-action@v1
2525
with:
2626
anthropic_api_key: ${{ secrets.ANTHROPIC_API_KEY }}
2727
track_progress: true # ✨ Enables tracking comments
2828
prompt: |
2929
REPO: ${{ github.repository }}
30-
PR NUMBER: ${{ github.event.issue.number || github.event.pull_request.number }}
30+
PR NUMBER: ${{ github.event.issue.number }}
3131
32-
Please review this pull request checked out in the current working directory with a focus on:
32+
Please review this pull request with a focus on:
3333
- Code quality and best practices
3434
- Potential bugs or issues
3535
- Security implications
3636
- Performance considerations
3737
38+
Note: The PR branch is already checked out in the current working directory.
39+
3840
Make sure to:
39-
- Use `gh pr comment` for top-level feedback
41+
- Use `gh pr comment` for concise top-level feedback
4042
- Use `mcp__github_inline_comment__create_inline_comment` to highlight specific code issues
43+
- Put all detailed feedback and recommendations in inline comments and do not duplicate in the main PR comment
44+
- Anywhere you have a line number reference, use an inline comment
4145
- Check structure is consist with existing code style
4246
- Skip style issues already handled by linters
4347
- Check for duplication of code already in the repository; aim for re-use where sensible

0 commit comments

Comments
 (0)