Skip to content

chore: group Dependabot updates per dependency across directories - #1827

Merged
nickclyde merged 1 commit into
mainfrom
nickclyde/dependabot-grouping-fixes
Oct 28, 2026
Merged

nickclyde merged 1 commit into
mainfrom
nickclyde/dependabot-grouping-fixes

Conversation

@nickclyde

@nickclyde nickclyde commented Oct 28, 2026 •

Copy link
Copy Markdown
Member

PULL REQUEST

Summary

Follow-up to #1809. After that merged, the multi-directory entries opened a separate PR per directory for the same update, plus one PR per constraint bump in seed-scripts:

Changes:

  • Adds a per-dependency group (group-by: dependency-name) to the npm, pip, and docker entries. It takes whatever the minor-and-patch group doesn't, so each major arrives as one PR covering every directory that uses it.
  • Stops pip version updates for containers/ecr-viewer/seed-scripts. Its requirements.txt only has >= floors, so Dependabot can't classify the bumps and opened one PR per release. Security updates still apply, and its Dockerfiles stay covered.

Related Issue

Follow-up to #1809

Acceptance Criteria

  • A major that applies to several directories arrives as one PR
  • No more version-update PRs for seed-scripts/requirements.txt

Additional Information

Dependabot reruns as soon as this lands on main. It should replace #1815/#1820 and #1816/#1819 with one PR each. The four seed-scripts PRs (#1823 to #1826) are for a directory that's no longer configured, so it's safe to close them.

I validated the config with check-jsonschema --builtin-schema vendor.dependabot.

Checklist

  • If this code affects the other scrum team, have they been notified? (In Slack, as reviewers, etc.) (n/a, Dependabot config only)

Related PRs

The same grouping fix is going into the other DIBBs repos. Merging them one at a time keeps each Dependabot rerun reviewable:

On their first run the multi-directory entries opened a separate PR per
directory for the same update (typescript 7 and eslint-plugin-jsdoc 65
each twice), plus a single PR per constraint bump in seed-scripts.

- Add a per-dependency group (group-by: dependency-name) to the npm,
  pip, and docker entries. It takes whatever the minor-and-patch group
  doesn't, so each major gets one PR covering every directory.
- Stop pip version updates for containers/ecr-viewer/seed-scripts. Its
  requirements.txt only has >= floors, so every release produced its own
  PR. Security updates still apply, and its Dockerfiles stay covered.

@robertmitchellv robertmitchellv left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

great job 🏆

@nickclyde
nickclyde enabled auto-merge (squash) October 28, 2026 21:14
@nickclyde
nickclyde merged commit edb94c5 into main Oct 28, 2026
20 checks passed
@nickclyde
nickclyde deleted the nickclyde/dependabot-grouping-fixes branch October 28, 2026 21:15
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.

2 participants