Skip to content
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

Code formatting #2

Open
PrzemoF opened this issue Aug 20, 2017 · 6 comments
Open

Code formatting #2

PrzemoF opened this issue Aug 20, 2017 · 6 comments

Comments

@PrzemoF
Copy link
Contributor

PrzemoF commented Aug 20, 2017

Please consider using python code auto formatting tools, like autopep8 [1].
[1] https://pypi.python.org/pypi/autopep8

@edwardvmills
Copy link
Owner

that looks really convenient....but time spent learning new tools is time not spent making new functionality. i have to walk a fine line. i'll definitely look into it, but no promises as far as implementation :)

@PrzemoF
Copy link
Contributor Author

PrzemoF commented Aug 21, 2017

I call it "sharpening the tools" :-) If it looks interesting I'll prepare a demo fork and you'll be able to pull the changes easily.

PEP8 uses spaces instead of tabs - is it OK for you?

P.S. All it takes is (on linux):
find . -name "*.py" | xarg -I [] autopep8 -i []
which means: find all python files in the project directory and call autopep8-in-place for every single one.

@edwardvmills
Copy link
Owner

is it available on windows? i use Notepad++ and i was hoping to see an option to do that. i'm concerned about relying on tools i don't have easy access to.

@PrzemoF
Copy link
Contributor Author

PrzemoF commented Aug 21, 2017

Looks like it is, but I don't use windows for writing code (vim+syntastic+pathogen+autopep8 is my setup and it all happens in the background after a proper setup)
Take a look here for notepad++ "plugin" : https://gist.github.com/kmonsoor/11374912

@luzpaz
Copy link
Contributor

luzpaz commented Mar 25, 2018

@edwardvmills i think this is a good idea. It makes reading code easier if you want others to collaborate, IMHO.
ref: #6

@luzpaz
Copy link
Contributor

luzpaz commented Mar 16, 2020

PR to add LGTM badges #15

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants