Skip to content

deps: consolidate six Dependabot bumps and group future updates - #161

Merged
ivanball merged 1 commit into
mainfrom
deps/consolidated-dependabot-sweep
Jul 28, 2026
Merged

deps: consolidate six Dependabot bumps and group future updates#161
ivanball merged 1 commit into
mainfrom
deps/consolidated-dependabot-sweep

Conversation

@ivanball

Copy link
Copy Markdown
Owner

Replaces six of the seven open Dependabot PRs with one, and adds grouping config so this stops recurring.

What ships

Package From To Was
actions/cache 4 6 #146
Azure.Monitor.OpenTelemetry.AspNetCore 1.5.0 1.6.0 #148
AngleSharp 1.5.2 1.6.0 #149
bunit 2.7.2 2.8.6 #150
SQLitePCLRaw.bundle_e_sqlite3 3.0.4 3.0.5 #153
System.IdentityModel.Tokens.Jwt 8.19.2 8.21.0 #154

Built by regeneration, not by merging the branches. Merging them would have conflicted: they touch overlapping packages.lock.json files (AngleSharp and bunit hit the same two; the analyzers PR touches all 26), and every merge re-BEHINDs the rest under "require branches to be up to date". Applying the version lines and running one dotnet restore --force-evaluate sidesteps that, and costs one CI run instead of six.

Deliberately NOT included: #147 (analyzers)

SonarAnalyzer.CSharp 10.29 → 10.30 ships two new rules, S8969 and S8970, which fail this build with 41 errors across 11 files under TreatWarningsAsErrors:

76 error S8969  (redundant null-forgiving operator)
 6 error S8970  (null-forgiving where nullable warnings are disabled)

The fixes are mechanical (drop a redundant !), but that is analyzer remediation, not a dependency bump, and bundling 41 unrelated source edits into a version-bump PR would make both harder to review and to revert. #147 stays open for its own PR.

Grouping config, so this stops recurring

dependabot.yml already grouped analyzers and opentelemetry — which is why two of the seven arrived pre-grouped. Two more groups close the gap:

  • testing — keeps bUnit and its AngleSharp dependency in one PR instead of two that touch the same lock files.
  • minor-and-patch — catch-all so ungrouped minor/patch updates arrive as one weekly PR. Majors stay ungrouped: those deserve their own read.

This matters because of what a Common PR costs: 13 required checks including a MAUI windows build, three browser a11y legs, a BenchmarkDotNet gate and a Testcontainers Redis tier.

Verification

  • Full solution Release: 2483/2483 green, 0 warnings, 0 errors.
  • 15 lock files regenerated.

Not a deploy: Common merges are not releases. These reach ADC/Store at the next version sweep.

Replaces six of the seven open Dependabot PRs with one, and adds the grouping
config so this stops recurring.

Bumps (#146, #148, #149, #150, #153, #154):
  actions/cache                             4        -> 6
  Azure.Monitor.OpenTelemetry.AspNetCore    1.5.0    -> 1.6.0
  AngleSharp                                1.5.2    -> 1.6.0
  bunit                                     2.7.2    -> 2.8.6
  SQLitePCLRaw.bundle_e_sqlite3             3.0.4    -> 3.0.5
  System.IdentityModel.Tokens.Jwt           8.19.2   -> 8.21.0

Built by applying the version lines to Directory.Packages.props and running ONE
dotnet restore --force-evaluate, rather than merging the six branches. Merging
them would have conflicted: they touch overlapping packages.lock.json files
(AngleSharp and bunit hit the same two), and every merge re-BEHINDs the rest
under "require branches to be up to date". Regenerating sidesteps that entirely.

NOT included: #147 (analyzers group). SonarAnalyzer 10.29 -> 10.30 ships two new
rules, S8969 and S8970, which fail this build with 41 errors across 11 files
under TreatWarningsAsErrors. Those are mechanical (remove a redundant
null-forgiving operator) but they are analyzer remediation, not a dependency
bump, and they belong in their own reviewable PR. #147 stays open.

dependabot.yml gains two groups. "testing" keeps bUnit and its AngleSharp
dependency in one PR instead of two that touch the same lock files.
"minor-and-patch" is a catch-all so ungrouped minor/patch updates arrive weekly
as one PR: this repo has 13 required checks including a MAUI windows build,
three browser a11y legs, a BenchmarkDotNet gate and a Testcontainers Redis tier,
so N separate dependency PRs cost N full re-runs. Majors stay ungrouped, since
those deserve their own read.

Full solution 2483/2483 green in Release, 0 warnings; 15 lock files regenerated.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01RpWirEitGZ2AMEQhd7NCD9
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