Skip to content

dependabot: group uv dev tooling and runtime patch/minor bumps - #1214

Open
potiuk wants to merge 1 commit into
apache:mainfrom
potiuk:dependabot-group-uv-deps
Open

dependabot: group uv dev tooling and runtime patch/minor bumps#1214
potiuk wants to merge 1 commit into
apache:mainfrom
potiuk:dependabot-group-uv-deps

Conversation

@potiuk

@potiuk potiuk commented Aug 28, 2026

Copy link
Copy Markdown
Member

Code change

Summary

Grouping was already in place for github/codeql-action*, but the uv ecosystem had none - so five separate PRs landed today (#1208-#1212) for ruff, mypy and types-pyyaml across /pelican and /stash. All five are PEP 735 dev group tooling that never ships in the published action, so one PR per tool buys no review value.

  • dev-tooling group (dependency-type: development, patterns: ["*"]) - collects the lint/test tooling. Dependabot opens one PR per directory per group, so today's five become two.
  • runtime-minor-patch group (dependency-type: production, minor + patch only) - keeps routine runtime bumps together.
  • Major bumps of runtime dependencies match no group and so keep their own PR. A major version of something the action ships can break consumers and deserves to be reviewed and released alone.
  • The allow-list ecosystem stays ungrouped, and now carries a comment saying why: every bump there gets its own security review and its own verify run, so grouping would tie an action that fails verification to unrelated ones that passed.
  • README documents the grouping table and both deliberate exclusions.

Dependabot classifies these correctly today - #1155 (markdown) reports dependency-type: direct:production and #1190 (ruff) reports direct:development - so both groups match what is already being emitted.

Type of change

  • Workflow or CI change
  • Documentation update

Testing

  • .github/dependabot.yml parses and resolves to the three expected ecosystems with the intended group keys.
  • prek run --all-files clean.
  • No code paths touched; the effect is visible on Dependabot's next weekly uv run.

Drafted-by: Claude Code (Opus 5); reviewed by @potiuk before posting

Five separate PRs landed on 2026-08-28 (apache#1208-apache#1212) for ruff, mypy and
types-pyyaml across /pelican and /stash. They are all PEP 735 dev-group
tooling that never ships in the published action, so reviewing them one
per tool buys nothing. Dependabot opens one PR per directory per group,
so this collapses them to one per directory.

Runtime dependencies are grouped for patch and minor only - a major bump
of something the action ships stays outside every group and keeps its own
PR. The allow-list ecosystem stays ungrouped and now says why in a comment.

Generated-by: Claude Code (Opus 5)
@potiuk
potiuk force-pushed the dependabot-group-uv-deps branch from 5303cc4 to c91756b Compare August 28, 2026 16:05
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