Skip to content

Give up on cross-repo fork PR reviews#458

Merged
4ab814ef-2a43-45d9-ad73-07758638023c merged 1 commit into
mainfrom
bugfix/give-up-on-forks
Oct 7, 2025
Merged

Give up on cross-repo fork PR reviews#458
4ab814ef-2a43-45d9-ad73-07758638023c merged 1 commit into
mainfrom
bugfix/give-up-on-forks

Conversation

@ar9708

@ar9708 ar9708 commented Oct 7, 2025

Copy link
Copy Markdown
Contributor

I tried here, ar9708#1, and here, ar9708#2, but even when the ANTHROPIC_API_KEY is picked up, it does nothing good.

Let us give up and start making the PRs in branches on OwnTube/web-client upstream instead for a while. Ping @mykhailodanilenko FYI!

I tried here, ar9708#1, and here, ar9708#2, but even when the `ANTHROPIC_API_KEY` is picked up, it does nothing good.

Let us give up and start making the PRs in branches on OwnTube/web-client upstream instead for a while. Ping @mykhailodanilenko FYI!

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR removes the Claude PR review GitHub workflow that was previously configured to automatically review pull requests using Anthropic's Claude API. The removal is motivated by issues with cross-repository fork PR reviews not functioning properly despite having the API key configured.

  • Removes the entire Claude review workflow file
  • Eliminates automated PR review functionality
  • Simplifies the repository by removing non-functional automation

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@ar9708 ar9708 mentioned this pull request Oct 7, 2025
15 tasks
@4ab814ef-2a43-45d9-ad73-07758638023c 4ab814ef-2a43-45d9-ad73-07758638023c merged commit 1d910f9 into main Oct 7, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants