Skip to content

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

Merged
Vincent Biret (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
Vincent Biret (baywet) merged 1 commit into
support/v2from
fix/null-to-v2

Conversation

@baywet

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
@github-code-quality

Copy link
Copy Markdown

Code Coverage Overview

Languages: C#

C# / code-coverage/dotnet

The overall coverage in the fix/null-to-v2 branch is 41%. The coverage in the support/v2 branch is 88%.

Show a code coverage summary of the most impacted files.
File support/v2 0ace243 fix/null-to-v2 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
Vincent Biret (baywet) merged commit bc11356 into support/v2 Jul 13, 2026
17 checks passed
@baywet
Vincent Biret (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