Skip to content

Fix analyzer RCS1250: Don't suggest a collection expression for the Dictionary - #1652

Merged
josefpihrt merged 4 commits into
dotnet:mainfrom
aihnatiuk-ccc:RCS1250-collection-expression-fix
Jul 26, 2025
Merged

josefpihrt merged 4 commits into
dotnet:mainfrom
aihnatiuk-ccc:RCS1250-collection-expression-fix

Conversation

@aihnatiuk-ccc

Copy link
Copy Markdown
Contributor

Fixes #1641

Hello, I am new to this project and to the Roslyn in general, this is my first contribution here.
I hope I got the idea right.

@aihnatiuk-ccc

Copy link
Copy Markdown
Contributor Author

@dotnet-policy-service agree

@aihnatiuk-ccc
aihnatiuk-ccc force-pushed the RCS1250-collection-expression-fix branch from 03cb36e to ceaa8dc Compare May 8, 2025 09:53
Comment thread ChangeLog.md Outdated
@aihnatiuk-ccc
aihnatiuk-ccc force-pushed the RCS1250-collection-expression-fix branch from ceaa8dc to 6df228e Compare May 26, 2025 07:41
@aihnatiuk-ccc
aihnatiuk-ccc requested a review from josefpihrt May 26, 2025 07:47
@aihnatiuk-ccc

Copy link
Copy Markdown
Contributor Author

@josefpihrt Something is wrong with the pipeline. Do not think this is my fault but correct me if I'm wrong. Could you rerun please?

@josefpihrt

Copy link
Copy Markdown
Collaborator

I rerun it, it should help.

Comment thread ChangeLog.md Outdated
@josefpihrt
josefpihrt merged commit 646f429 into dotnet:main Jul 26, 2025
@josefpihrt

Copy link
Copy Markdown
Collaborator

@aihnatiuk Thanks for the contribution! 👍

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.

RCS1250 fixer generates invalid syntax for Dictionaries

2 participants