Skip to content

chore(deps): ignore major NuGet bumps (keep minor/patch + security) - #229

Merged
thomasluizon merged 1 commit into
mainfrom
chore/dependabot-security-only
Jun 23, 2026
Merged

thomasluizon merged 1 commit into
mainfrom
chore/dependabot-security-only

Conversation

@thomasluizon

Copy link
Copy Markdown
Owner

Change

The backend isn't Expo-locked, so NuGet minor/patch version updates stay useful — but major bumps (e.g. Stripe.net 50→52) are breaking and need manual review + billing tests, not an auto-PR.

  • nuget → keep the grouped minor/patch version updates; ignore all majors (dependency-name: "*", version-update:semver-major). Scoping the ignore to majors keeps Dependabot security updates flowing.
  • github-actions → grouped weekly version updates.

Paired with the npm security-only change in orbit-ui-mobile (its mobile workspace is Expo-SDK-pinned and install-managed).

🤖 Generated with Claude Code

… + security)

Major NuGet bumps (e.g. Stripe.net 50->52) are breaking and need manual review + testing, not auto-PRs. Ignore version-update:semver-major globally so only grouped minor/patch version updates open automatically; scoping the ignore to major version-updates leaves Dependabot security updates intact. GitHub Actions grouped.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This is a clean Dependabot configuration hardening: NuGet major-version bumps are now suppressed via version-update:semver-major (the correct scoping string, which leaves security advisories unaffected per Dependabot's documented behavior), and GitHub Actions updates are collapsed into a single grouped PR each week. The YAML structure is valid, the ignore and groups blocks are complementary and non-conflicting, and the removal of the explicit open-pull-requests-limit on the actions entry is safe because grouping keeps steady-state at 1 PR and the GitHub default of 5 applies as a backstop. No correctness, security, contract, or CLAUDE.md issues found.

@thomasluizon
thomasluizon merged commit b31eaa5 into main Jun 23, 2026
6 checks passed
@thomasluizon
thomasluizon deleted the chore/dependabot-security-only branch June 23, 2026 21:02
@sonarqubecloud

Copy link
Copy Markdown

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