Skip to content

ci: skip automatic PR description drafts for bot PRs - #283

Merged
markbackman merged 1 commit into
mainfrom
ci/skip-bot-pr-descriptions
Nov 4, 2026
Merged

markbackman merged 1 commit into
mainfrom
ci/skip-bot-pr-descriptions

Conversation

@markbackman

Copy link
Copy Markdown
Contributor

The Draft PR Description workflow has Claude write a PR's description when the PR is opened. It fails on every Dependabot PR (for example #278), so those PRs show a red check that has nothing to do with the dependency bump. This change makes the automatic trigger skip PRs opened by bots.

  • Automatic trigger. Dependabot pushes its branches to this repo, so the existing fork check doesn't filter them out. claude-code-action then stops with Workflow initiated by non-human actor: dependabot (type: Bot). The job's if: now also requires github.event.pull_request.user.type != 'Bot', so on these PRs the job is skipped instead of failing.
  • Why not allowed_bots. Workflows that Dependabot triggers don't get the repository's Actions secrets, so ANTHROPIC_API_KEY is empty in these runs. Allowing the bot would only move the failure to the next step.
  • Manual triggers. The /pr-description and /release-notes comments and workflow_dispatch work the same as before.

This is the matching edit for the copy in pipecat-ai/pipecat-client-web-transports#262, which the workflow file says to keep in sync.

Dependabot PRs are same-repo, so the fork guard didn't catch them, and
claude-code-action rejects bot actors. They also don't get repository
secrets, so allowing the bot wouldn't help.

@mattieruth mattieruth 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.

Image

@markbackman
markbackman merged commit f22f467 into main Nov 4, 2026
4 checks passed
@markbackman
markbackman deleted the ci/skip-bot-pr-descriptions branch November 4, 2026 15:59
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.

2 participants