Skip to content

Bump the code-quality group with 1 update - #313

Closed
dependabot[bot] wants to merge 1 commit into
mainfrom
dependabot/nuget/code-quality-ec160c8502
Closed

Bump the code-quality group with 1 update#313
dependabot[bot] wants to merge 1 commit into
mainfrom
dependabot/nuget/code-quality-ec160c8502

Conversation

@dependabot

@dependabot dependabot Bot commented on behalf of github Jul 27, 2026

Copy link
Copy Markdown
Contributor

Updated SonarAnalyzer.CSharp from 10.29.0.143774 to 10.30.0.144632.

Release notes

Sourced from SonarAnalyzer.CSharp's releases.

10.30.0.144632

Release notes - .NET Analyzers - 10.30

Feature

NET-1536 Implement rule S8970: Null-forgiving operators should not be used when nullable warnings are disabled
NET-3436 Implement rule S8949: Use the overload that accepts a CancellationToken
NET-3810 Fix: Protobuf Importer logs debug on excluded files
NET-3877 Implement rule S8747: Migrations should not narrow column types without converting existing data
NET-4091 Implement rule S8969: Null-forgiving operators should not be redundant
NET-4120 Update RSPEC before 10.30 release

False Positive

NET-1541 Fix S3459 FP: support classes marked with [AutoConstructor] attribute
NET-1583 Fix S6967 FP: Raises when model has no validation attributes
NET-1840 Fix S3903 FP: top-level statements and partial Program in separate file
NET-4059 Improve precision of S8949 (CancellationTokenShouldBeUsed) - umbrella
NET-4191 Fix S3169 FP: Should not raise in Azure Cosmos

False Negative

NET-3819 Fix S1244 FN: Should report on Double.Equals

Bug

NET-4107 Fix S4026 Race Condition

Commits viewable in compare view.

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore <dependency name> major version will close this group update PR and stop Dependabot creating any more for the specific dependency's major version (unless you unignore this specific dependency's major version or upgrade to it yourself)
  • @dependabot ignore <dependency name> minor version will close this group update PR and stop Dependabot creating any more for the specific dependency's minor version (unless you unignore this specific dependency's minor version or upgrade to it yourself)
  • @dependabot ignore <dependency name> will close this group update PR and stop Dependabot creating any more for the specific dependency (unless you unignore this specific dependency or upgrade to it yourself)
  • @dependabot unignore <dependency name> will remove all of the ignore conditions of the specified dependency
  • @dependabot unignore <dependency name> <ignore condition> will remove the ignore condition of the specified dependency and ignore conditions

Bumps SonarAnalyzer.CSharp from 10.29.0.143774 to 10.30.0.144632

---
updated-dependencies:
- dependency-name: SonarAnalyzer.CSharp
  dependency-version: 10.30.0.144632
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: code-quality
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot Bot added .NET Pull requests that update .NET code dependencies Pull requests that update a dependency file labels Jul 27, 2026
@vgmello

vgmello commented Jul 27, 2026

Copy link
Copy Markdown
Owner

Not merging: SonarAnalyzer.CSharp 10.30 introduces new rules that fail the build on existing code, incl. a false-positive S8969 conflicting with compiler nullability (CS8620). Held at 10.29; see #315.

@vgmello vgmello closed this Jul 27, 2026
@dependabot @github

dependabot Bot commented on behalf of github Jul 27, 2026

Copy link
Copy Markdown
Contributor Author

This pull request was built based on a group rule. Closing it will not ignore any of these versions in future pull requests.

To ignore these dependencies, configure ignore rules in dependabot.yml

@dependabot
dependabot Bot deleted the dependabot/nuget/code-quality-ec160c8502 branch July 27, 2026 14:08
vgmello added a commit that referenced this pull request Jul 27, 2026
…Tools, Scalar) (#315)

Combines the pending Dependabot dependency PRs into one, superseding
**#308, #309, #310, #311, #314** (and closing the held-back **#312,
#313**).

## Included
- **WolverineFx** + `.Kafka` + `.Postgresql` + `.RuntimeCompilation`
`6.21.0` → `6.23.0` (#309, #314). `.Kafka` aligned manually —
Dependabot's `wolverine` group only bumped 3 of the 4, which would have
left Kafka a minor behind its siblings.
- **Microsoft.Orleans** `10.2.1` → `10.2.2` (#308)
- **Grpc.Tools** `2.82.0` → `2.83.0` (#310)
- **Scalar.AspNetCore** `2.16.15` → `2.16.16` (#311)

## Intentionally excluded
- **Refitter.MSBuild 2.1.0** (#312) — generator breaks E2E client
generation (`Method not found:
System.Text.ValueStringBuilder.AsSpan()`).
- **SonarAnalyzer.CSharp 10.30** (#313) — new analyzer rules fail the
build on existing code, including a false-positive **S8969** that
conflicts with the compiler's own nullability analysis (removing the
flagged `!` triggers CS8620).

## Verification
- `dotnet build AppDomain.slnx` — 0 warnings / 0 errors.
- `dotnet test AppDomain.Tests` — 209/209.
- **Generated-template E2E**: `dotnet new mmt --local` (full config,
Wolverine 6.23), booted against real Postgres + Kafka, ran the generated
E2E suite → **14/14 passed**.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file .NET Pull requests that update .NET code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant