-
Notifications
You must be signed in to change notification settings - Fork 677
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Added ability to add parameter description attributes #1202
base: master
Are you sure you want to change the base?
Conversation
Some unit tests will be nice... |
…ersion. Added unit test
Ref work done in AspNetCore version domaindrivendev/Swashbuckle.AspNetCore#554 |
@jsheely |
After thinking about it using Perhaps there could be an option to enable using [Description] for anyone who wants that. |
Here is a good reason to use [SwaggerDescription] instead of [Description]. Now I can add summary to the heading of each method. |
See the build log:
|
I think the compiler does not like your short syntax: |
Any idea how to update the AppVeyor compiler to do the right thing? I'd rather not have to change the operator. I really like the null check operator. |
What version of Visual Studio are you using? I don't think that was available back then... |
Well I made the build work. But I have no idea why appVeyor doesn't build the same way as it does locally. Should be using Visual Studio 2017. |
I recommend to explicitly set C# language version in project properties -> build -> advanced. The default settings uses latest available C# version. which depends on version of Visual Studio you (or the build server) are using. |
There was no way to add descriptions to query param without using XML comments.