feat: #2114 - add required field management in csharp newtonsoft serializer - Githubissues

asyncapi / modelina

A library for generating typed models based on inputs such as AsyncAPI, OpenAPI, and JSON Schema documents with high customization

https://modelina.org

Apache License 2.0

322 stars 184 forks source link

feat: #2114 - add required field management in csharp newtonsoft serializer #2117

Open fr-th opened 3 weeks ago

fr-th commented 3 weeks ago

Description

When generating c# model with newtonsoft serialization, an option now allows to prevent deserialization if required fields are missing

Related Issue

https://github.com/asyncapi/modelina/issues/2114

Checklist

[x] The code follows the project's coding standards and is properly linted (npm run lint).
[x] Tests have been added or updated to cover the changes.
[x] Documentation has been updated to reflect the changes.
[x] All tests pass successfully locally.(npm run test).

Additional Notes

netlify[bot] commented 3 weeks ago

Deploy Preview for modelina canceled.

Name	Link
Latest commit	7bf5f908e038b1d990a80afb8b7f6d1ad61e814a
Latest deploy log	https://app.netlify.com/sites/modelina/deploys/6720bbc393667100087e9832

sonarcloud[bot] commented 3 weeks ago

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

coveralls commented 3 weeks ago

Pull Request Test Coverage Report for Build 11572166800

Details

3 of 3 (100.0%) changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.009%) to 92.305%

Totals
Change from base Build 11517599539:	-0.009%
Covered Lines:	6004
Relevant Lines:	6335

💛 - Coveralls

fr-th commented 3 weeks ago

Ah damn, I missed the part for the 'next' branch sorry. I'll adapt