Skip to content

Conversation

@AlekseyTs
Copy link
Contributor

The member isn't visited by default, but NullableWalker visits it.

Fixes #81552.

…sionSpreadElement.Conversion`

The member isn't visited by default, but `NullableWalker` visits it.
@AlekseyTs AlekseyTs requested a review from a team as a code owner December 8, 2025 22:04
@jcouv jcouv self-assigned this Dec 8, 2025
Copy link
Member

@jcouv jcouv left a comment

Choose a reason for hiding this comment

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

LGTM Thanks (commit 1)

@AlekseyTs AlekseyTs merged commit 332f407 into dotnet:main Dec 8, 2025
26 checks passed
@dotnet-policy-service dotnet-policy-service bot added this to the Next milestone Dec 8, 2025
333fred added a commit to 333fred/roslyn that referenced this pull request Dec 9, 2025
* upstream/main: (221 commits)
  [main] Update dependencies from dotnet/roslyn (dotnet#81601)
  Update Roslyn.Diagnostics.Analyzers (dotnet#81599)
  Remove unnecessary lightup code
  Remove unnecessary lightup code
  Remove unnecessary lightup code
  Remove unnecessary lightup code
  Remove unnecessary lightup code
  Build
  NullableWalker.DebugVerifier: Explicitly visit `BoundCollectionExpressionSpreadElement.Conversion` (dotnet#81584)
  SyntheticBoundNodeFactory: Remove ability to create arbitrary conversions  (dotnet#81569)
  Docs
  Tweak
  Remove dependency on Newtonsoft.Json from Microsoft.CodeAnalysis.Workspaces.MSBuild.BuildHost (dotnet#81562)
  Simplify
  Apply suggestions from code review
  Make into theory
  Updates
  Add tests
  Add tests
  Improve go-to-def navigation in overload resolution failure cases
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

NullableWalker assert failure with a collection expression passed to params collection

3 participants