Skip to content

Hold Repl packages to patch bumps only - #410

Merged
erikdarlingdata merged 1 commit into
devfrom
fix/dependabot-repl-pin
Jul 25, 2026
Merged

Hold Repl packages to patch bumps only#410
erikdarlingdata merged 1 commit into
devfrom
fix/dependabot-repl-pin

Conversation

@erikdarlingdata

Copy link
Copy Markdown
Owner

Holds the Repl packages that arrived with #388 to patch bumps only.

Why this is not just noise control

Repl, Repl.Mcp and Repl.Testing landed at 0.11.0 - pre-1.0, where semver permits a minor bump to break.

The important detail: Dependabot classifies 0.11 -> 0.12 as a minor, and this config groups minor and patch into one batch while giving majors their own PR. So the existing "majors stay separate" protection does nothing here - a breaking Repl bump would arrive folded inside a routine grouped dependency PR, which is exactly the kind of thing that gets merged on a green check.

It matters more than for a typical dependency because Performance Studio ships an auto-updater. Whatever lands reaches users without them choosing to take it.

What it does

Ignores semver-major and semver-minor for all three packages. Patch bumps (0.11.x) still flow, so genuine fixes and any security patches arrive normally. Mirrors the existing ignore rules for the other deliberate pins.

Note

This is independent of whether the Repl pilot itself ships. The packages are on dev either way, and Dependabot now targets dev (fixed in #409), so it would start proposing these on its next scheduled run regardless.

Test plan

  • dependabot.yml parses; all nine ignore rules resolve as intended, both ecosystems still target-branch: dev
  • Config-only change - no build impact

🤖 Generated with Claude Code

#388 brought Repl, Repl.Mcp and Repl.Testing onto dev at 0.11.0. They
are pre-1.0, where a minor bump is allowed to break under semver - and
Dependabot classifies 0.11 -> 0.12 as a *minor*, so the config's usual
"majors get their own PR, minor/patch are grouped" split gives no
protection at all here. A breaking bump would arrive inside a routine
grouped batch.

That matters more than for a typical dependency because the app ships an
auto-updater, so whatever lands reaches users without them choosing it.

Ignores semver-major and semver-minor for the three packages; patch
bumps still flow so genuine fixes arrive. Mirrors the existing ignore
rules for the other deliberate pins.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@claude

claude Bot commented Jul 25, 2026

Copy link
Copy Markdown

Reviewed. Config-only change to .github/dependabot.yml.

Verified the package names match exactly what's referenced in the tree: Repl and Repl.Mcp in src/PlanViewer.Cli/PlanViewer.Cli.csproj, Repl.Testing in tests/PlanViewer.Core.Tests/PlanViewer.Core.Tests.csproj, all at 0.11.0 as the PR states. The three new ignore entries are correctly nested inside the nuget ecosystem block (before the github-actions block starts), and follow the same shape as the existing Avalonia*/Microsoft.VisualStudio.SDK major-only pins, just extended to also ignore semver-minor — consistent with the stated pre-1.0 rationale. YAML indentation and structure look correct.

No correctness, security, or repo-convention issues here — this doesn't touch app code, T-SQL generation, or plan-XML parsing. LGTM.

@erikdarlingdata
erikdarlingdata merged commit fdd038e into dev Jul 25, 2026
3 checks passed
@erikdarlingdata
erikdarlingdata deleted the fix/dependabot-repl-pin branch July 25, 2026 23:31
This was referenced Jul 26, 2026
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