Skip to content

fix: handle nullability more accurately during serialization for 3.0/2.0 (#2933)#2937

Merged
baywet merged 1 commit into
support/v2from
fix/null-to-v2
Jul 13, 2026
Merged

fix: handle nullability more accurately during serialization for 3.0/2.0 (#2933)#2937
baywet merged 1 commit into
support/v2from
fix/null-to-v2

Conversation

@baywet

@baywet baywet commented Jul 13, 2026

Copy link
Copy Markdown
Member

port of #2933 to v2

…2.0 (#2933)

* Handle nullability more accurately

* Cleanup

* Rename SerializeTypeProperty to TrySerializeTypeProperty

* Separate rp

* Cleanup

* Progress

* Cleanp

* Refactor

* Use cached json extension

* AddExtension only if Type is still unknown

* Cleanup

* Address review comments

* Address review comments

* Update OpenApiSchemaDeserializer.cs

* Update OpenApiSchemaDeserializer.cs
@baywet baywet requested a review from a team as a code owner July 13, 2026 17:58
@baywet baywet self-assigned this Jul 13, 2026
@baywet baywet enabled auto-merge July 13, 2026 17:58
@github-code-quality

Copy link
Copy Markdown

Code Coverage Overview

Languages: C#

C# / code-coverage/dotnet

The overall coverage in the branch is 41%. The coverage in the branch is 88%.

Show a code coverage summary of the most impacted files.
File 0ace243 310b6e2 +/-
/home/runner/wo...MapParameter.cs 100% 0% -100%
/home/runner/wo...Deserializer.cs 81% 0% -81%
/home/runner/wo...pyReferences.cs 63% 31% -32%
/home/runner/wo...piJsonReader.cs 100% 72% -28%
/home/runner/wo...Deserializer.cs 90% 62% -28%
/home/runner/wo...s/OpenApiTag.cs 100% 76% -24%
/home/runner/wo...Deserializer.cs 100% 81% -19%
/home/runner/wo...Deserializer.cs 96% 81% -15%
/home/runner/wo...Deserializer.cs 97% 89% -8%
/home/runner/wo...piComponents.cs 94% 87% -7%

Code Coverage is in Public Preview. Learn more and provide us with your feedback.

@sonarqubecloud

Copy link
Copy Markdown

@baywet baywet merged commit bc11356 into support/v2 Jul 13, 2026
17 checks passed
@baywet baywet deleted the fix/null-to-v2 branch July 13, 2026 18:24
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.

3 participants