Skip to content

fix(ci): check PR author instead of actor for dependabot detection#34

Merged
jmgilman merged 1 commit into
masterfrom
fix/go-mod-tidy-condition
Jan 20, 2026
Merged

fix(ci): check PR author instead of actor for dependabot detection#34
jmgilman merged 1 commit into
masterfrom
fix/go-mod-tidy-condition

Conversation

@jmgilman
Copy link
Copy Markdown
Contributor

Summary

Fixes the go-mod-tidy workflow skipping on Dependabot PRs.

Problem

The workflow was using github.actor == 'dependabot[bot]' to detect Dependabot PRs. However, github.actor is the user who triggered the current event, not necessarily who opened the PR.

When someone synchronizes a Dependabot PR (or when any subsequent event occurs), the actor changes to that user, causing the job to skip.

Solution

Use github.event.pull_request.user.login instead, which checks who opened the PR.

Test plan

  • Re-run workflow on existing Dependabot PR to confirm it no longer skips

🤖 Generated with Claude Code

The workflow was skipping because `github.actor` is the user who
triggered the event (e.g., someone synchronizing the PR), not
necessarily dependabot.

Use `github.event.pull_request.user.login` to check who opened the PR.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented Jan 20, 2026

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
blob 650f34d Commit Preview URL

Branch Preview URL
Jan 20 2026, 05:26 AM

@jmgilman jmgilman merged commit d767490 into master Jan 20, 2026
7 checks passed
@jmgilman jmgilman deleted the fix/go-mod-tidy-condition branch January 20, 2026 05:28
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