Skip to content

ci: don't fail fork PRs on the LLM cost comment - #2160

Merged
assafelovic merged 1 commit into
mainfrom
ci/fork-safe-cost-check
Sep 26, 2026
Merged

assafelovic merged 1 commit into
mainfrom
ci/fork-safe-cost-check

Conversation

@assafelovic

Copy link
Copy Markdown
Owner

llm-costs.yml failed on every fork PR with "Resource not accessible by integration": fork PRs get a read-only GITHUB_TOKEN, and the action posts a PR comment. #1821, for example, shows a red check while all of its tests pass.

The job is now continue-on-error for forks only. The analysis still runs and is visible in the job log, and same-repo PRs behave exactly as before.

actionlint clean.

🤖 Generated with Claude Code

The cost analysis job failed on every fork PR ("Resource not accessible
by integration"): fork PRs get a read-only token and the action posts a
PR comment. Mark the job continue-on-error for forks only, so the
analysis still runs and a fork PR's checks reflect its tests.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@assafelovic
assafelovic merged commit 869030f into main Sep 26, 2026
10 checks passed
@assafelovic
assafelovic deleted the ci/fork-safe-cost-check branch September 26, 2026 13:25
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.

1 participant