python-uds icon indicating copy to clipboard operation
python-uds copied to clipboard

Refactor to proper PEP standards

Open RoninDusette opened this issue 5 years ago • 1 comments

While for sure a great piece of work, it needs a major refactor for legibility. It currently "pisses into the wind" of the standards in place for formatting functions/methods, classes, spacing is inconsistent, as much as it can be for a space delineated language. It sucks that it would need it, but would likely only take a weekend to do so. I am guessing you came from a Java background (just because of the formatting).

I am not talking trash, I am just saying that most IDEs will autoformat most of it to the standards of the language in context. Go with it, not against.

RoninDusette avatar May 24 '20 02:05 RoninDusette

I appreciate the feedback :) I started on C so you had a good first shot.

I'll definitely put it on the list of improvements as I'm seeing some more traffic on this project than I used to.

richClubb avatar May 26 '20 07:05 richClubb