Repository navigation
ci: let Dependabot label all its pull requests - #406
Conversation
Remove `target-branch` from every Dependabot entry: it names the default branch anyway, and while it is set, GitHub doesn't apply an entry's options, labels included, to security updates. Security update pull requests then get `type: dependencies` instead of Dependabot's own labels. The `ignore` rules now also apply to security updates, as intended. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Reviewer's GuideConfigurationRemove
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## dev #406 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 28 28
Lines 3517 3517
=========================================
Hits 3517 3517 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Linked issuesChecked at commit cfbac6d #405 ci: let Dependabot label all its pull requests · 5 of 6 met · 1 to check by hand
|
There was a problem hiding this comment.
Hey - I've reviewed your changes and they look great!
Sourcery assessment
Needs a human reviewer. Security-update PRs may now receive the configured labels, while the target branch remains the default branch as described in the config comment. Reverting stops future label changes, but labels already applied to open PRs remain until removed; that cleanup is bounded and straightforward.
Summary
Lets Dependabot label all its pull requests, security updates included, with the organisation's label
type: dependenciesonly, instead of its own labels (dependenciesplus an ecosystem label).In
.github/dependabot.yml:target-branch: devfrom every entry (4 entries).devis the default branch, so version updates still target it.target-branchwith one that explains why no entry sets it.ignorerules. They now also apply to security updates: theconnexionmajor hold, andfocaandelixircloud/foca, which are bumped by hand together with the base image (a lone bump breaks the image build), so Dependabot won't open security updates for FOCA either. That's intended.Nothing else changes: compared as data (keys sorted), the new file equals the old one minus
target-branchpluslabelson every entry.Closes #405
Light lane: CI configuration only (Dependabot), no secrets, permissions or release workflow touched, so the reviewer subagents were skipped; I reviewed the diff myself.
Verification
yqparses the new.github/dependabot.yml; everyupdatesentry haslabels: ["type: dependencies"]; notarget-branchis left.yq -o=json 'sort_keys(..)'): only the intended change.CI passedgreen) after mergingdev, which brought the dnspython fix of build(deps): bump dnspython from 2.9.0 to 2.9.2 #404 (CVE-2026-128089 had failedScan imagebefore).Declined review findings
None.
Open questions
None.
🤖 Generated with Claude Code
Summary by Sourcery
Enhancements:
type: dependencieslabel consistently to Dependabot security and version-update pull requests by removing explicit target branches from all update configurations.