Skip to content

#268 ci: use a valid pull-request-branch-name separator in dependabot config - #269

Merged
RAprogramm merged 1 commit into
mainfrom
268
Aug 24, 2026
Merged

#268 ci: use a valid pull-request-branch-name separator in dependabot config#269
RAprogramm merged 1 commit into
mainfrom
268

Conversation

@RAprogramm

@RAprogramm RAprogramm commented Aug 24, 2026

Copy link
Copy Markdown
Owner

Closes #268

The Dependabot config check still failed after #267:

The property '#/updates/0/pull-request-branch-name/separator' value "/-" did not match one of the following values: -, /, _

"/-" was never a valid separator; Dependabot fell back to the default /, which is what existing branches already use. Setting / explicitly keeps branch naming unchanged and makes the file schema-valid.

Verified locally: the whole file validates against the official Dependabot 2.0 JSON schema with zero errors; zizmor and actionlint unchanged.

Summary by CodeRabbit

  • Chores
    • Updated automated dependency update branch naming for Cargo and GitHub Actions changes.

@RAprogramm
RAprogramm enabled auto-merge (squash) August 24, 2026 01:43
@coderabbitai

coderabbitai Bot commented Aug 24, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 613160a7-1816-4786-9bd3-bbdec6f42030

📥 Commits

Reviewing files that changed from the base of the PR and between 4fe7dad and da06f11.

📒 Files selected for processing (1)
  • .github/dependabot.yml

Included review availability: Your plan provides up to 10 included reviews per hour; 6 remain after this review.


📝 Walkthrough

Walkthrough

The Dependabot configuration changes the Cargo and GitHub Actions branch-name separator from /- to /. This makes both values valid under the Dependabot schema.

Changes

Dependabot configuration

Layer / File(s) Summary
Update branch-name separators
.github/dependabot.yml
The Cargo and GitHub Actions update entries now use / as the pull-request branch-name separator.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to da06f

This localized configuration change makes the Dependabot branch separator schema-valid while preserving existing branch naming, so no actionable merge-blocking risk remains after normal checks and review.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly identifies the Dependabot separator correction and its purpose.
Linked Issues check ✅ Passed The change replaces "/-" with "/", preserves branch naming, and addresses the schema validation failure described in issue [#268].
Out of Scope Changes check ✅ Passed The changes are limited to the two Dependabot separator updates required by issue [#268].
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch 268

Comment @coderabbitai help to get the list of available commands.

@codecov

codecov Bot commented Aug 24, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@RAprogramm
RAprogramm merged commit 6c6e729 into main Aug 24, 2026
37 checks passed
@RAprogramm
RAprogramm deleted the 268 branch August 24, 2026 01:46
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 config rejects pull-request-branch-name separator "/-"

1 participant