feat: Enable AOT-compatibility#96
Conversation
… by using generic attributes
also return IList instead of IEnumerable everywhere for consistency
|
FYI: I will need a few days for the review. I'm pretty busy this week. |
|
I'll do the review this weekend. |
|
No worries. This currently uses a preview version of xunit aot. Stable release should be soon. |
HofmeisterAn
left a comment
There was a problem hiding this comment.
Thanks, the PR looks good! I just have a few questions so I can understand a few things better.
…zer / new JsonRequestContent
…alize(object) call
|
The analyser is working now and also found some more missing types. I tried to make it generic (which means I merged the converter into the attributes) and update the analyser but somehow it didn't work. The analyser didn't analyse generic attributes somehow (maybe because it is a new feature) or I did something wrong. |
If I got it right, you're referring to |
|
@HofmeisterAn Could you cut a new release? Would love to try this out! Amazing work :) |




IsAotCompatibleDockerModelsJsonSerializerContextwith all docker API models which have JSON parameters, so query string only models are not added for JsonSerializationDockerExtendedJsonSerializerContextwhich includes types directly used by *Operations (this can be removed / merged with DockerModelsJsonSerializerContext when STJ source generator fails when defining the serializer context in 2 partial classes dotnet/runtime#99669 is available)DynamicallyAccessedMembersattributesxunit.v3.aot.mtp-v24.0.0-pre.33which is still in preview but required to actually test AOT.xunit.v3.mtp-v24.0.0-pre.33for .NET 8.0 because xunit AOT only works from .NET 9 onwards.IEnumerableresponses to beIListfor consistency (breaking change)QueryStringParameterto reduce reflectionfixes #88