ci(deps): stop Dependabot proposing unmergeable and split-version bumps - #76
Merged
Merged
Conversation
Three open Dependabot PRs could never go green, for two distinct reasons. Microsoft.OpenApi (#75): Microsoft.AspNetCore.OpenApi 10.x constrains it to >=2.7.5 <3.0.0, so the proposed 3.10.2 resolves outside the constraint and raises NU1608 — an error here, not a warning, because Directory.Build.props sets TreatWarningsAsErrors. No 3.x can merge until ASP.NET Core moves first. Ignore major updates for that package only; 2.x patches still flow, which is what the explicit pin exists for (GHSA-v5pm-xwqc-g5wc). codeql-action (#72, #69): init and analyze are pinned to a single SHA, and Dependabot raised one PR per sub-action. Either alone lands a mismatch that fails at run time with "Loaded a configuration file for version X, but running version Y". The same hazard sits in #70/#68 — test-suite.yml uploads and downloads artifacts, so the two actions must share a major. Group both families ahead of the actions-all catch-all so co-versioned actions always travel in one PR. Groups match in definition order. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.Scanned FilesNone |
This was referenced Aug 24, 2026
bgard68
added a commit
that referenced
this pull request
Aug 24, 2026
upload-artifact 4.6.2 -> 7.0.1 and download-artifact 4.3.0 -> 8.0.1. Dependabot raised these as #70 and #68. test-suite.yml uploads coverage artifacts in two jobs and downloads them in a third, and the artifact backend is not compatible across these majors — merging either PR alone would leave main with a broken upload/download handoff until the other landed. Combining them keeps main consistent at every commit. The dependabot.yml grouping added in #76 makes this automatic from the next run; this commit clears the pair already in flight. Co-authored-by: Claude Opus 5 <noreply@anthropic.com>
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
1 participant
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.
Three open Dependabot PRs cannot go green. Two distinct causes, both fixed here in
.github/dependabot.yml.1.
Microsoft.OpenApi2.12.0 → 3.10.2 (#75) can never mergeMicrosoft.AspNetCore.OpenApi 10.0.11constrainsMicrosoft.OpenApito>= 2.7.5 && < 3.0.0. The proposed 3.10.2 resolves outside that constraint and raisesNU1608, which is an error rather than a warning becauseDirectory.Build.propssetsTreatWarningsAsErrors.Reproduced locally:
No 3.x can merge until ASP.NET Core itself depends on
Microsoft.OpenApi3.x. Major updates are now ignored for that one package; 2.x patches still flow, which is what the explicit pin exists for (GHSA-v5pm-xwqc-g5wc).2. Co-versioned actions were split across PRs (#72, #69 — and latent in #70, #68)
codeql-action/initandcodeql-action/analyzeare pinned to a single SHA, but Dependabot raised one PR per sub-action. Either one alone lands a version mismatch that only fails at run time:The same hazard is latent in #70 / #68:
test-suite.ymluploads artifacts and later downloads them, soupload-artifactanddownload-artifactmust stay on the same major. Merging one without the other breaks the handoff.Both families now have explicit groups placed before the
actions-allcatch-all, since groups match in definition order.Verification
dotnet build -c Release— succeeded, 0 warnings, 0 errorsdotnet testagainstpostgres:16-alpinematching the CI service — 348 passed, 0 failed (295 unit + 53 integration)NU1608; reverted.github/dependabot.ymlparses, and groups resolve in the intended orderFollow-up (not in this PR)
Close #75, #72 and #69 — they cannot go green as proposed. Dependabot will re-raise the CodeQL bump as one grouped PR on the next run. #70 and #68 should be merged together or not at all.