[THRIFT-3596] Better conformance to PEP8

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 0.10.0
• Component/s: Python - Library
• Labels: RFC
• Assignee: Nobuaki Sukegawa
• Reporter: Nobuaki Sukegawa
• Created: 02/Feb/16 19:55
• Updated: 28/Jan/19 06:17
• Resolved: 04/Feb/16 08:31

Description

py coding_standards.md states it follows PEP8 but currently it does not do so at all.
So a typical experience of a first-time (potential) py contributor would be to see red warnings all over the display and then has to either adjust their editor's settings or stop there.

On the other hand, a huge downside of global reformat is git-blame experience but will be mostly mitigated by git blame -w in this case.

Issue Links

• is duplicated by: THRIFT-2954 calling function with parameter type different as defined in idl breaks the connection
• is related to: THRIFT-3601 Better conformance to PEP8 for generated code