Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add support for nested non-nullable choice members #340

Merged
merged 2 commits into from
Jun 16, 2022

Conversation

jmatss
Copy link
Contributor

@jmatss jmatss commented Jun 15, 2022

Fixes #339.

Previously only direct children of choices would be forced to nullable (see how choices are handled in XmlSchemaClassGenerator). This pull request adds support to force nested descendants of choices to nullable if needed.

@codecov
Copy link

codecov bot commented Jun 15, 2022

Codecov Report

Merging #340 (dd4c63f) into master (8204bee) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #340      +/-   ##
==========================================
+ Coverage   95.53%   95.55%   +0.01%     
==========================================
  Files          19       19              
  Lines        3044     3057      +13     
  Branches      465      468       +3     
==========================================
+ Hits         2908     2921      +13     
  Misses         78       78              
  Partials       58       58              
Impacted Files Coverage Δ
XmlSchemaClassGenerator/ModelBuilder.cs 97.72% <100.00%> (+0.03%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8204bee...dd4c63f. Read the comment docs.

@mganss mganss merged commit 961b3a2 into mganss:master Jun 16, 2022
@mganss
Copy link
Owner

mganss commented Jun 16, 2022

Thanks!

@jmatss jmatss deleted the support-nested-choice-elements branch June 16, 2022 17:40
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Nested non-nullable elements in choice not supported
2 participants