Skip to content

Fix SA1649 for C# 15 union declarations - #4096

Merged
1 commit merged into
DotNetAnalyzers:masterfrom
verdie-g:fix/sa1649-union-declarations
Oct 7, 2026
Merged

1 commit merged into
DotNetAnalyzers:masterfrom
verdie-g:fix/sa1649-union-declarations

Conversation

@verdie-g

Copy link
Copy Markdown
Contributor

Fixes the SA1649 analyzer crash on C# 15 union declarations by handling all TypeDeclarationSyntax nodes.

Adds regression coverage for diagnostics, code fixes, and generic unions.

Tests: 119 SA1649 tests passed.

Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
@AArnott

AArnott commented Oct 7, 2026

Copy link
Copy Markdown
Contributor

Thank you @verdie-g for finding this crash and for the fix! Your commit is merged with its original authorship as part of #4127, which brings it onto the C# 15 test project that has since landed on master and also handles unnamed type declarations such as C# 14 extension blocks. Closing this one in favor of #4127.

@AArnott AArnott closed this pull request by merging all changes into DotNetAnalyzers:master in 12d7949 Oct 7, 2026
This was referenced Nov 11, 2026
This was referenced Nov 11, 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.

2 participants