Skip to content

fix: use pull_request.user.login instead of github.actor for bot check#2910

Merged
adamtheturtle merged 1 commit intomainfrom
fix/dependabot-actor-check
Mar 17, 2026
Merged

fix: use pull_request.user.login instead of github.actor for bot check#2910
adamtheturtle merged 1 commit intomainfrom
fix/dependabot-actor-check

Conversation

@adamtheturtle
Copy link
Member

@adamtheturtle adamtheturtle commented Mar 17, 2026

Fixes adamtheturtle/literalizer#146

The github.actor check is unreliable - it reflects who triggered the workflow, not who opened the PR. Use github.event.pull_request.user.login instead.

Flagged by zizmor bot-conditions audit.

Made with Cursor


Note

Low Risk
Small change to a GitHub Actions condition; risk is limited to whether Dependabot PRs are correctly detected and auto-merged.

Overview
Updates the Dependabot auto-merge GitHub Actions workflow to gate execution on the PR author (github.event.pull_request.user.login) instead of github.actor, avoiding cases where the workflow trigger user differs from the PR opener.

Adjusts zizmor.yml by removing the disabled bot-conditions rule now that the workflow condition is compliant.

Written by Cursor Bugbot for commit 15a170a. This will update automatically on new commits. Configure here.

@adamtheturtle adamtheturtle force-pushed the fix/dependabot-actor-check branch from d7b7db7 to 15a170a Compare March 17, 2026 09:13
@adamtheturtle adamtheturtle merged commit df731f0 into main Mar 17, 2026
16 checks passed
@adamtheturtle adamtheturtle deleted the fix/dependabot-actor-check branch March 17, 2026 11:18
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.

dependabot-merge.yml: use pull_request.user.login instead of github.actor for bot check

1 participant