NSwag icon indicating copy to clipboard operation
NSwag copied to clipboard

Added option to use standard C# naming conventions (and more)

Open kendallb opened this issue 4 years ago • 6 comments

Added options to use standard C# naming conventions as well as the ability to generate a client without the Async suffix on functions when Async is the only API method supported. Also trimmed new lines from all the .liquid files to avoid conflicts on dangling spaces in diffs in the future.

kendallb avatar Aug 07 '20 19:08 kendallb

Just checked in support for automatic rate limit handling (http status code 429) with the UI to optionally enable it.

kendallb avatar Aug 07 '20 21:08 kendallb

I believe when I did this, it was not possible to modify the naming generator in NJsonSchema to support this, because the generator stuff is all handles in NSwag itself? Or are you suggesting that the default naming convention in NJsonSchema be changed to use correct C# naming conventions and get rid of the options and overrides to change it?

kendallb avatar Sep 17 '20 22:09 kendallb

Or are you suggesting that the default naming convention in NJsonSchema be changed to use correct C# naming conventions and get rid of the options and overrides to change it?

Yes, i wonder how much we’d need to break?

RicoSuter avatar Sep 17 '20 22:09 RicoSuter

Well I am all for making it the default naming convention since the other way was wrong :) But I do wonder if other folks using it might do a double take if they can't go back to what it was before?

kendallb avatar Sep 17 '20 23:09 kendallb

are there any plans to resolve this? me and @christianerbsmehl would love to use nswag, but that naming issue is a showstopper for us.

fsamiec avatar Jul 27 '22 13:07 fsamiec

I would like to see this merged in also. We are still using our own custom build of NSwag as it works for the purposes we need it for with all my patches. You can use my tree and build against that as an alternative?

https://github.com/kendallb/NSwag

kendallb avatar Jul 27 '22 17:07 kendallb