Merge pull request #216 from alephao/master

yonaskolb / SwagGen

OpenAPI/Swagger 3.0 Parser and Swift code generator

MIT License

626 stars 146 forks source link

Closed alephao closed 4 years ago

alephao commented 4 years ago

This PR fixes the issue described on #215

Added an extension to Property to get the correct value for the nullable property.
Using Property.nullable instead of checking property.required and property.schema.metadata.nullable on SwiftFormatter and CodeFormatter
Added a test covering this case

yonaskolb commented 4 years ago

Great PR, thanks @alephao! Could you please add a changelog entry and then we can get this merged 👍

yonaskolb commented 4 years ago

Adding in master

alephao commented 4 years ago

Sorry, I was away during the weekend. Thanks!