Azure / autorest

OpenAPI (f.k.a Swagger) Specification code generator. Supports C#, PowerShell, Go, Java, Node.js, TypeScript, Python
MIT License
4.61k stars 737 forks source link

[Python] DocString should respect parameter order #1690

Closed lmazuel closed 7 years ago

lmazuel commented 7 years ago

Example, this is incorrect: image

The right order of parameters in the DocString is:

I guess it's because required parameters are moved to the start of the method signature. Created confusion with a customer (and me :) ).

FYI @annatisch

fearthecowboy commented 7 years ago

@lmazuel - is this still an issue? Reopen if it needs to be tracked.

lmazuel commented 7 years ago

@fearthecowboy Please re-open this one, I don't have the necessary permission to do it.

lmazuel commented 7 years ago

We reduced the number of parameters, making this problem disappeared by itself.