-
Notifications
You must be signed in to change notification settings - Fork 20
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Pep 8 Style Guide for Python Code #162
base: develop
Are you sure you want to change the base?
Conversation
Durables Support, Basic Semver, Tests
I am not the king of the hill. I'm happy to change code styles if they make sense. Maybe you can sell me on this idea that I personally hate. I like to use this style:
Other people like this (which I hate):
The thing I'm demonstrating is that the |
We do not include IDE specific files in our repos. In addition, before merging something like this we'd need to see if any of our other repos have adopted style-guides for python, and evaluate. @bobinson - I'd suggest creating an issue, outlining what it is you're trying to achieve with Pep 8 and how it makes tinman better than it is today. That'll give us something to review-from. I'm leaving this PR open, but it is not to be considered for merging until the |
@inertia186 & @relativityboy : I will revert with an issue report detailing why we need a generic coding style. Also removed the non-standard editor files which was there by mistake. Just to add to @inertia186's points, I am also not "religious" about a certain coding style over another one (for that matter, I am not even religious about Vim or Emacs). For example, in this very PR I had kept the following non PEP8 compliant section as I found it pretty neat ;-) |
We see the changes. Thanks. Doing some internal legwork to see if this is something we want to adopt. |
This PR brings in PEP-8 compatibility to the Tinman codebase.