heldersepu / Swagger-Net

Seamlessly adds a swagger to WebApi projects!
BSD 3-Clause "New" or "Revised" License
144 stars 42 forks source link

Update SchemaExtensions.cs #126

Closed scott-xu closed 1 year ago

scott-xu commented 1 year ago

Set schema's description only if has SwaggerDescriptionAttribute applied and the description is not null, in case overwrite the description from custom SchemaFilter or ModelFilter

scott-xu commented 1 year ago

@heldersepu could you please help review the PR? Thanks!

heldersepu commented 1 year ago

Ye, looks good if you can add some unit tests later

scott-xu commented 1 year ago

Thank you @heldersepu ! I'll add some unit tests. When do you think is ready to release a new version?

heldersepu commented 1 year ago

New release published https://www.nuget.org/packages/Swagger-Net/8.4.19.1