ENG-5551 - Add grouped weekly Dependabot config for daisybill_api - #134
Merged
baueric merged 1 commit intoAug 18, 2026
Merged
Conversation
Adds .github/dependabot.yml for the bundler and github-actions ecosystems on a weekly Monday schedule. Groups are split production/development and restricted to update-types: [minor, patch] so breaking major bumps arrive as individual PRs instead of blocking the whole group. open-pull-requests-limit is 5 per ecosystem. Format matches the config in daisyBill/daisybill. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
baueric
force-pushed
the
eng-5551-add-grouped-weekly-dependabot-config-for-
branch
from
August 17, 2026 11:53
ed5e346 to
fcd8ad9
Compare
Contributor
Author
|
@maximsilver another one for dependabot grouped PRs |
There was a problem hiding this comment.
Pull request overview
Adds a Dependabot configuration to enable scheduled, grouped dependency updates for this repository (Bundler + GitHub Actions), aligning with the goal of making dependency maintenance and alert remediation actionable.
Changes:
- Introduces
.github/dependabot.ymlwith weekly (Monday 10:00) update checks. - Configures grouped PRs for Bundler (split prod vs dev) and GitHub Actions, restricted to minor/patch updates.
- Caps open Dependabot PRs at 5 per ecosystem to avoid PR noise.
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
maximsilver
approved these changes
Aug 18, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes ENG-5551
Problem
This repo had no
.github/dependabot.ymlandautomated-security-fixeswas disabled outright (enabled=false, paused=false— off deliberately, not auto-paused). The "0 open Dependabot PRs" figure was misleading: nothing was running.There are 9 open alerts on
Gemfile.lock(2 high, 4 medium, 3 low):This is also the only public repo in the ENG-5537 set. Advisories here are visible to anyone, and anyone depending on the gem inherits them.
Change
Adds
.github/dependabot.ymlcovering both ecosystems present in the repo:/) and github-actions (/, for.github/workflows/ruby.yml)schedule.interval: weekly, Monday 10:00open-pull-requests-limit: 5per ecosystemupdate-types: [minor, patch]Format deliberately mirrors the config in
daisyBill/daisybill(minus the private-registry and per-gemignoreblocks, which don't apply here).Why
update-typesfrom the startPer ENG-5558: the daisyBill config (PR #19078) omitted
update-typesand both grouped PRs it produced failed CI immediately. Restricting groups to minor/patch means breaking majors arrive as individual PRs that can be triaged on their own rather than blocking the whole group.Lockfile review is required on every grouped PR here
update-typesconstrains only what Dependabot updates directly — transitive resolution is unconstrained. On daisyBill (ENG-5564) a transitiverack2→3 forced a silentsinatradowngrade to 1.0 that broke Rails boot, and ajquery-uiminor bump lifted transitivejqueryto 4.0.0 with CI fully green.Neither
racknorjqueryis present here, so those two specific guards don't apply. But this is a published gem — any resolution change in its lockfile is inherited by consumers, so read theGemfile.lockdiff on every grouped PR, not just the first.Gem is alive — configure, don't disable
ENG-5551 asked whether the gem is still published and consumed, since disabling alerts (ENG-5539) would be more honest than configuring updates nobody merges.
It's live:
daisybill_apiv0.2.0 published 2026-03-24, 245 downloads on that version, 24,883 all-time, not yanked. Configuring updates is the right call.Known limitation: stale gemspec constraints block some patches
Two alerts can't be fixed by a minor/patch group because the gemspec pins below the patched version:
rake "~> 10.0"— needs>= 12.3.3(major bump)bundler "~> 2.0"/ others are fine;yard "~> 0.9.0"does reach 0.9.44Dependabot will surface
rakeas an individual major-version PR. Widening that constraint is a separate call and out of scope for this PR.Follow-up (not in this PR)
Dependabot security updates still need enabling on the repo — a settings change, not a file:
Deploy Note
Infrastructure — adds
.github/dependabot.ymlonly. No gem code change, no version bump, no release.Product Review
None