sort keyword arguments

[onetonfoot]

Sort the keyword arguments when generating python code #83

[coveralls]

Pull Request Test Coverage Report for Build 326

• 1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage increased (+0.02%) to 79.367%

---

Totals
Change from base Build 323:	0.02%
Covered Lines:	2251
Relevant Lines:	2721

---

💛 - Coveralls

[stevematthew]

Hi, this change would be useful for me too.

Anything i can do to help get this merged in?

[dossett]

Hi @sambogoat thanks for this PR (and your other one as well), I will try to look next week. Having schools closed this fall has made a couple part of my job (like open source work) hard to prioritize. We really appreciate your interest and potential contribution!

[stevematthew]

Hi @dossett any chance you would have bandwidth to review this PR and #91 too?

Thanks in advance.

[dossett]

@onetonfoot I discussed with my team and we agree this is a useful feature, but did not necessarily want to make it the default behavior. If you could make it controllable and add some unit tests, that would be great. I will promptly review any additional work you (or anyone) does on this PR. Thank you!

[dossett]

Changes requested in comments

[onetonfoot]

Sorry I don't have time to continue working on this.

[jdimatteo]

Deterministic behavior is always preferable from my point of view and having randomly ordered keyword arguments is problematic for automated integration tests -- I wish this work would get revived at some point!