Skip to content

fix(ci): allow dependabot PRs through fork-PR guard - #25

Merged
hummbl-dev merged 2 commits into
mainfrom
fix/devin/allow-dependabot-ci
Aug 9, 2026
Merged

fix(ci): allow dependabot PRs through fork-PR guard#25
hummbl-dev merged 2 commits into
mainfrom
fix/devin/allow-dependabot-ci

Conversation

@hummbl-dev

Copy link
Copy Markdown
Collaborator

Summary

  • Add github.event.pull_request.user.login == 'app/dependabot' to the fork-PR guard in ci.yml
  • Dependabot PRs are trusted (created by GitHub's dependabot bot) but were being skipped by the fork-PR guard, resulting in zero CI checks
  • This is safe because dependabot only modifies dependency manifests — it cannot inject arbitrary code

Context

Origin: 2026-08-09 AAR — governed-compression #24 (dependabot GitHub Action bump) had zero CI checks and was merged via admin without any validation.

Test plan

  • Next dependabot PR on this repo gets CI checks

Generated with Devin

@hummbl-dev
hummbl-dev force-pushed the fix/devin/allow-dependabot-ci branch from 7ea0594 to cd4efc8 Compare August 9, 2026 21:25
@hummbl-dev
hummbl-dev merged commit aefdbca into main Aug 9, 2026
2 checks passed
@hummbl-dev
hummbl-dev deleted the fix/devin/allow-dependabot-ci branch August 9, 2026 21:27
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