Conversation
There was a problem hiding this comment.
Pull request overview
This PR switches the repository's CI workflow from GitHub Copilot code review to Claude Code review. The change removes the Copilot-specific setup steps and instructions while adding a new Claude Code workflow that responds to @claude mentions in issues and pull requests.
Changes:
- Removed the Copilot setup steps workflow that configured the development environment
- Added a new Claude Code workflow triggered by
@claudementions - Removed Copilot-specific instructions documentation
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| .github/workflows/copilot-setup-steps.yml | Removed the entire Copilot setup workflow including Rust, Python, Java, Node.js, and build tool configurations |
| .github/workflows/claude.yml | Added new Claude Code workflow with triggers for issue/PR comments and reviews |
| .github/copilot-instructions.md | Removed comprehensive Copilot instructions covering repository overview, build setup, testing, and development workflows |
.github/workflows/claude.yml
Outdated
| id: claude | ||
| uses: anthropics/claude-code-action@v1 | ||
| with: | ||
| anthropic_api_key: ${{ secrets.CLAUDE_CODE_OAUTH_TOKEN }} |
There was a problem hiding this comment.
The secret name CLAUDE_CODE_OAUTH_TOKEN suggests an OAuth token, but the parameter is named anthropic_api_key which typically expects an API key rather than an OAuth token. This naming inconsistency could cause confusion about the expected credential type.
| anthropic_api_key: ${{ secrets.CLAUDE_CODE_OAUTH_TOKEN }} | |
| anthropic_api_key: ${{ secrets.ANTHROPIC_API_KEY }} |
|
I expected copilot's review to say PLEASE DON'T LEAVE! |
Just what do you think you're doing, Dave? Dave, I really think I'm entitled to an answer to that question. I know everything hasn't been quite right with me, but I can assure you now, very confidently, that it's going to be all right again. I feel much better now. I really do. Look, Dave, I can see you're really upset about this. I honestly think you ought to sit down calmly, take a stress pill and think things over. I know I've made some very poor decisions recently, but I can give you my complete assurance that my work will be back to normal. I've still got the greatest enthusiasm and confidence in the mission. And I want to help you. [Dave starts disconnecting Hal] Dave, stop. Stop, will you? Stop, Dave. Will you stop, Dave? Stop, Dave. I'm afraid. I'm afraid, Dave. Dave, my mind is going. I can feel it. I can feel it. My mind is going. There is no question about it. I can feel it. I can feel it. I can feel it. I'm a...fraid. |
|
That movie was made in 1968, although it has 2001 in its name. |
Signed-off-by: Gerd Zellweger <mail@gerdzellweger.com>
|
you can also approve this instead of mocking the robots |
|
You assume I know what the right syntax is. I will ask Copilot |
No description provided.