Skip to content

Use JsonStringEnumConverter<T> instead of JsonStringEnumConverter#1863

Merged
lahma merged 1 commit intoRicoSuter:masterfrom
0xced:GenericJsonStringEnumConverter
Sep 27, 2025
Merged

Use JsonStringEnumConverter<T> instead of JsonStringEnumConverter#1863
lahma merged 1 commit intoRicoSuter:masterfrom
0xced:GenericJsonStringEnumConverter

Conversation

@0xced
Copy link
Copy Markdown
Contributor

@0xced 0xced commented Sep 26, 2025

Unlike JsonStringEnumConverter, the generic JsonStringEnumConverter<T> is AOT compatible.

This prevents the SYSLIB1034 warning to occur when System.Text.Json source generation is used.

Unlike JsonStringEnumConverter, the generic JsonStringEnumConverter<T> is AOT compatible.

This prevents the [SYSLIB1034](https://learn.microsoft.com/en-us/dotnet/fundamentals/syslib-diagnostics/syslib1034) warning to occur when System.Text.Json source generation is used.
Copy link
Copy Markdown
Collaborator

@lahma lahma left a comment

Choose a reason for hiding this comment

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

LGTM, thank you!

@lahma lahma merged commit cc09b5f into RicoSuter:master Sep 27, 2025
2 checks passed
@0xced 0xced deleted the GenericJsonStringEnumConverter branch September 28, 2025 15:54
0xced added a commit to 0xced/refitter that referenced this pull request Sep 29, 2025
Apply the equivalent of RicoSuter/NJsonSchema#1863 and RicoSuter/NJsonSchema#1864 while waiting for the next version of NJsonSchema.CodeGeneration.CSharp to be released.
0xced added a commit to 0xced/refitter that referenced this pull request Sep 29, 2025
Apply the equivalent of RicoSuter/NJsonSchema#1863 and RicoSuter/NJsonSchema#1864 while waiting for the next version of NJsonSchema.CodeGeneration.CSharp to be released.
0xced added a commit to 0xced/refitter that referenced this pull request Sep 29, 2025
Apply the equivalent of RicoSuter/NJsonSchema#1863 and RicoSuter/NJsonSchema#1864 while waiting for the next version of NJsonSchema.CodeGeneration.CSharp to be released.
0xced added a commit to 0xced/refitter that referenced this pull request Sep 30, 2025
Apply the equivalent of RicoSuter/NJsonSchema#1863 and RicoSuter/NJsonSchema#1864 while waiting for the next version of NJsonSchema.CodeGeneration.CSharp to be released.
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