adamjones1

Results 18 comments of adamjones1

Yeah it's a difficult problem. It goes broader than nulls too, generally it's the issue of interpreting untagged unions with potentially overlapping types. Eg. `int | long`, `int | int`,...

Was this applied to the `SwaggerClientProvider` as well as the `OpenApiClientProvider`? I still seem to be getting an `HttpRequestException` thrown. I tried migrating to `OpenApiClientProvider` instead but it wouldn't compile...

> if I got it correctly, it looks like by design in OpenAPI.NET =( microsoft/OpenAPI.NET#198 (comment) Urgh, that's annoying. > It is possible in your case to update names in...

Yeah, probably a preferred and more functional way would be to have a union type returned like ``` type MyOperationResult = | Success of MyOperationSuccessSchema | NotFound of MyOperationNotFoundSchema |...

@NickDarvey Yeah true...it does feel a bit hacky to me though and I don't feel great about it, sneaking an exception in the handler pipeline to divert control flow and...

I've only just noticed this thread but FYI in case it helps, I've had a PR open in NJsonSchema for a long time which aims to add this feature and...

When I use `references: strict`, including in the above repro, I don't get this behaviour - it takes no effect and all transitive dependencies are referenced as usual, like below:...

I took a further look at this today and thinking about it a little more, it must be a fundamental limitation of NuGet PackageReferences that `references: strict` can't ever work...