Skip to content

ci: exclude Dependabot PRs from auto-code review#2910

Merged
michaeldwan merged 1 commit intomainfrom
md/exclude-dependabot-from-bonk
Apr 7, 2026
Merged

ci: exclude Dependabot PRs from auto-code review#2910
michaeldwan merged 1 commit intomainfrom
md/exclude-dependabot-from-bonk

Conversation

@michaeldwan
Copy link
Copy Markdown
Member

Dependabot PRs are failing the Bonk auto-code review workflow because Dependabot doesn't have write permissions to the repository. This happens when the workflow triggers on Dependabot PRs (for example, when a maintainer with write permissions asks Dependabot to rebase).

The fix adds a condition to skip the review job when the PR author is Dependabot.

@michaeldwan michaeldwan requested a review from a team as a code owner April 7, 2026 16:15
@ask-bonk
Copy link
Copy Markdown
Contributor

ask-bonk bot commented Apr 7, 2026

LGTM

github run

@michaeldwan
Copy link
Copy Markdown
Member Author

1 line change to unfuck CI on dependabot PRs. merging without review to get things moving

@michaeldwan michaeldwan merged commit 507f88c into main Apr 7, 2026
37 of 38 checks passed
@michaeldwan michaeldwan deleted the md/exclude-dependabot-from-bonk branch April 7, 2026 16:17
markphelps added a commit that referenced this pull request Apr 8, 2026
…icate/cog into mphelps/push-phase-progress

* 'mphelps/push-phase-progress' of https://github.com/replicate/cog: (95 commits)
  feat: add metric name validation (#2911)
  Rename `cog run` to `cog exec` (#2916)
  chore(deps): bump github.com/google/go-containerregistry (#2884)
  fix: replace removed libgl1-mesa-glx in tensorflow integration test (#2914)
  ci: enforce stub freshness in CI, fix existing stub drift (#2912)
  feat: add schema-compare command to test harness (#2891)
  chore(deps): bump uuid from 1.22.0 to 1.23.0 in /crates (#2887)
  chore(deps): bump github.com/hashicorp/go-version from 1.7.0 to 1.9.0 (#2909)
  chore(deps): bump insta from 1.46.3 to 1.47.2 in /crates (#2908)
  fix: support list[X] | None inputs + integration tests for PEP 604 union File/Path coercion (#2882)
  ci: exclude Dependabot PRs from auto-code review (#2910)
  chore(deps): bump actions/checkout from 4 to 6 (#2904)
  chore(deps): bump github.com/testcontainers/testcontainers-go/modules/registry (#2886)
  fix: metrics bugs in coglet prediction server (#2896)
  Bump version to 0.17.2 (#2903)
  fix(coglet): propagate metric scope to async event loop thread (#2902)
  chore: remove unnecessary nolint directive in test (#2803)
  feat(coglet): add Sentry error reporting for infrastructure errors (#2865)
  fix: homebrew cask postflight xattr references wrong binary name (#2899)
  fix: include custom metrics in cog predict --json output (#2897)
  ...
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