Skip to content

ci: let Dependabot PRs auto-merge when CodeQL is neutral - #1265

Merged
dprevost-LMI merged 1 commit into
webdriverio:mainfrom
dprevost-LMI:ci/auto-merge-allow-neutral
Oct 8, 2026
Merged

dprevost-LMI merged 1 commit into
webdriverio:mainfrom
dprevost-LMI:ci/auto-merge-allow-neutral

Conversation

@dprevost-LMI

Copy link
Copy Markdown
Contributor

The auto-merge job accepted only success and skipped checks, but the CodeQL check always ends as neutral. So the job always failed, also when all tests passed (for example #1262), and every Dependabot PR was merged by hand. Allow neutral, which is not a failure; canceled or failed checks still block the auto-merge.

The auto-merge job accepted only success and skipped checks. CodeQL
default setup does not analyze commits pushed by Dependabot, so on a
Dependabot PR the `CodeQL` check ends as `neutral` ("2 configurations
not found"). The job always failed, also when all tests passed (for
example webdriverio#1262), and every Dependabot PR was merged by hand.

Allow `neutral`, which is not a failure. `main` is analyzed by CodeQL
after the merge, and canceled or failed checks (also a CodeQL finding)
still block the auto-merge.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@changeset-bot

changeset-bot Bot commented Oct 8, 2026 •

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 3c8fa42

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@dprevost-LMI
dprevost-LMI force-pushed the ci/auto-merge-allow-neutral branch from 512f6ab to 3c8fa42 Compare October 8, 2026 00:42
@dprevost-LMI
dprevost-LMI marked this pull request as ready for review October 8, 2026 00:45
@greptile-apps

greptile-apps Bot commented Oct 8, 2026 •

Copy link
Copy Markdown

RetriggerConfidence Score: 5/5

[High risk] Changes CI auto-merge rules for dependency updates.

The PR appears safe to merge; no actionable issues were found.

Summary

Allows Dependabot auto-merge to proceed when a check finishes as neutral.

  • Keeps success and skipped allowed.
  • Keeps failed and canceled checks outside allowed-conclusions.
  • Leaves the existing npm and non-major update limits unchanged.
  • No actionable issues found.

Reviews (1) · Last reviewed commit: "ci: let Dependabot PRs auto-merge when C..." · Reviewed by Greptile

@dprevost-LMI
dprevost-LMI merged commit 930eb4d into webdriverio:main Oct 8, 2026
10 checks passed
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