Add Claude Code GitHub Workflow #3011
Conversation
tonidero
left a comment
tonidero
left a comment
There was a problem hiding this comment.
Looking good! Just acouple questions. And we should also look into reusing the actions/workflows
| # if: | | ||
| # github.event.pull_request.user.login == 'external-contributor' || | ||
| # github.event.pull_request.user.login == 'new-developer' || | ||
| # github.event.pull_request.author_association == 'FIRST_TIME_CONTRIBUTOR' |
There was a problem hiding this comment.
Hmm do we need to filter out external contributors/forks here?
.github/workflows/claude.yml
Outdated
| actions: read | ||
| # Optional: Give a custom prompt to Claude. If this is not specified, Claude will perform the instructions specified in the comment that tagged it. | ||
| # prompt: 'Update the pull request description to include a summary of changes.' |
There was a problem hiding this comment.
This is a bit confusing... we're not giving it write access right?
There was a problem hiding this comment.
right... I am confused as well. This was generated btw, but for that it would require the write permission above this
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #3011 +/- ##
=======================================
Coverage 78.99% 78.99%
=======================================
Files 341 341
Lines 13500 13500
Branches 1825 1825
=======================================
Hits 10664 10664
Misses 2078 2078
Partials 758 758 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
| # Only allow org members and owners to trigger Claude reviews | ||
| if: | | ||
| github.event.pull_request.author_association == 'MEMBER' || | ||
| github.event.pull_request.author_association == 'OWNER' |
There was a problem hiding this comment.
What does OWNER mean here? PR owner? Maybe MEMBER is enough, as everyone will be RC org member?
There was a problem hiding this comment.
Owner of the repo. I asume owners are members? I'll double check
There was a problem hiding this comment.
I added them both just in case owners were not members, but it looks like they are so I'll just do member
| on: | ||
| issue_comment: | ||
| types: [created] | ||
| pull_request_review_comment: | ||
| types: [created] | ||
| issues: | ||
| types: [opened, assigned] | ||
| pull_request_review: | ||
| types: [submitted] |
There was a problem hiding this comment.
What's the difference between this workflow and claude-code-review.yml? Do we need both?
There was a problem hiding this comment.
we probably don't need the claude.yml one. It's to be able to tag claude in issues and PR comments, to request reviews if I understand correctly. The other one runs automatically
|
@claude what can you do? |
|
I've moved this to RevenueCat/sdk-github-workflows#2 so it can be shared everywhere |
|
Reopened to reuse the shared workflow |
🤖 Installing Claude Code GitHub App
This PR adds a GitHub Actions workflow that enables Claude Code integration in our repository.
What is Claude Code?
Claude Code is an AI coding agent that can help with:
How it works
Once this PR is merged, we'll be able to interact with Claude by mentioning @claude in a pull request or issue comment.
Once the workflow is triggered, Claude will analyze the comment and surrounding context, and execute on the request in a GitHub action.
Important Notes
Security
There's more information in the Claude Code action repo.
After merging this PR, let's try mentioning @claude in a comment on any PR to get started!